diff options
author | norly <ny-git@enpas.org> | 2013-06-25 16:07:03 +0100 |
---|---|---|
committer | norly <ny-git@enpas.org> | 2013-06-25 16:07:03 +0100 |
commit | e67148322089591dad4814c2ab3d66f728e1ecc6 (patch) | |
tree | fe6d709ecf7559c33ffcf792eadee675f313badc /src/libelfu/elfops | |
parent | 1d3855bd4296654d2f2c89fd5aefdc66cdda2a42 (diff) |
Check for (and abort on) multiple symbol tables
Diffstat (limited to 'src/libelfu/elfops')
-rw-r--r-- | src/libelfu/elfops/check.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/src/libelfu/elfops/check.c b/src/libelfu/elfops/check.c index e184b26..108964a 100644 --- a/src/libelfu/elfops/check.c +++ b/src/libelfu/elfops/check.c @@ -162,6 +162,18 @@ int elfu_eCheck(Elf *e) ELFU_WARN("elfu_eCheck: Sections %d and %d overlap in file.\n", i, j); goto ERROR; } + + /* We may not have more than one symbol table */ + if (shdrs[i].sh_type == SHT_SYMTAB && shdrs[j].sh_type == SHT_SYMTAB) { + ELFU_WARN("elfu_eCheck: Found more than one SYMTAB section.\n"); + goto ERROR; + } + + /* We may not have more than one dynamic symbol table */ + if (shdrs[i].sh_type == SHT_DYNSYM && shdrs[j].sh_type == SHT_DYNSYM) { + ELFU_WARN("elfu_eCheck: Found more than one DYNSYM section.\n"); + goto ERROR; + } } /* Section addr/offset should match parent PHDR. @@ -172,7 +184,7 @@ int elfu_eCheck(Elf *e) if (shdrs[i].sh_offset != shoff || !PHDR_CONTAINS_SCN_IN_FILE(&phdrs[j], &shdrs[i])) { - ELFU_WARN("elfu_eCheck: Memory/file offsets/sizes are not congruent for SHDR %d, PHDR %d.\n", i, j); + ELFU_WARN("elfu_eCheck: SHDR %d and PHDR %d report conflicting file/memory regions.\n", i, j); goto ERROR; } } |