From 9372319dd3f5150992893728bf74e28bb2b78d1f Mon Sep 17 00:00:00 2001 From: Alain Magloire Date: Tue, 13 Jan 2004 20:32:54 +0000 Subject: [PATCH] We have to check the range of the st_shndx field before using it some values are reserved. --- .../org/eclipse/cdt/utils/elf/ElfHelper.java | 39 ++++++++++++------- 1 file changed, 24 insertions(+), 15 deletions(-) diff --git a/core/org.eclipse.cdt.core/utils/org/eclipse/cdt/utils/elf/ElfHelper.java b/core/org.eclipse.cdt.core/utils/org/eclipse/cdt/utils/elf/ElfHelper.java index c10ac1157bc..b4d91e7d0e0 100644 --- a/core/org.eclipse.cdt.core/utils/org/eclipse/cdt/utils/elf/ElfHelper.java +++ b/core/org.eclipse.cdt.core/utils/org/eclipse/cdt/utils/elf/ElfHelper.java @@ -123,10 +123,13 @@ public class ElfHelper { for (int i = 0; i < dynsyms.length; i++) { if (dynsyms[i].st_bind() == Elf.Symbol.STB_GLOBAL && dynsyms[i].st_type() == Elf.Symbol.STT_FUNC) { int idx = dynsyms[i].st_shndx; - if (idx < 0) - continue; - if (sections[idx].sh_type == Elf.Section.SHT_NULL) + if (idx < Elf.Symbol.SHN_HIPROC && idx > Elf.Symbol.SHN_LOPROC) { + String name = dynsyms[i].toString(); + if (name != null && name.trim().length() > 0) + v.add(dynsyms[i]); + } else if (idx >= 0 && sections[idx].sh_type == Elf.Section.SHT_NULL) { v.add(dynsyms[i]); + } } } @@ -144,11 +147,13 @@ public class ElfHelper { for (int i = 0; i < dynsyms.length; i++) { if (dynsyms[i].st_bind() == Elf.Symbol.STB_GLOBAL && dynsyms[i].st_type() == Elf.Symbol.STT_OBJECT) { int idx = dynsyms[i].st_shndx; - if (idx < 0) - continue; - - if (sections[idx].sh_type == Elf.Section.SHT_NULL) + if (idx < Elf.Symbol.SHN_HIPROC && idx > Elf.Symbol.SHN_LOPROC) { + String name = dynsyms[i].toString(); + if (name != null && name.trim().length() > 0) + v.add(dynsyms[i]); + } else if (idx >= 0 && sections[idx].sh_type == Elf.Section.SHT_NULL) { v.add(dynsyms[i]); + } } } @@ -181,11 +186,13 @@ public class ElfHelper { for (int i = 0; i < symbols.length; i++) { if (symbols[i].st_bind() == Elf.Symbol.STB_GLOBAL && symbols[i].st_type() == Elf.Symbol.STT_FUNC) { int idx = symbols[i].st_shndx; - if (idx < 0) - continue; - - if (sections[idx].sh_type != Elf.Section.SHT_NULL) + if (idx < Elf.Symbol.SHN_HIPROC && idx > Elf.Symbol.SHN_LOPROC) { + String name = symbols[i].toString(); + if (name != null && name.trim().length() > 0) + v.add(symbols[i]); + } else if (idx >= 0 && sections[idx].sh_type != Elf.Section.SHT_NULL) { v.add(symbols[i]); + } } } @@ -203,11 +210,13 @@ public class ElfHelper { for (int i = 0; i < symbols.length; i++) { if (symbols[i].st_bind() == Elf.Symbol.STB_GLOBAL && symbols[i].st_type() == Elf.Symbol.STT_OBJECT) { int idx = symbols[i].st_shndx; - if (idx < 0) - continue; - - if (sections[idx].sh_type != Elf.Section.SHT_NULL) + if (idx < Elf.Symbol.SHN_HIPROC && idx > Elf.Symbol.SHN_LOPROC) { + String name = symbols[i].toString(); + if (name != null && name.trim().length() > 0) + v.add(symbols[i]); + } else if (idx >= 0 && sections[idx].sh_type != Elf.Section.SHT_NULL) { v.add(symbols[i]); + } } }