Fix courgette ELF x86 dissembler

courgette/disassembler_elf_32.cc

Index: courgette/disassembler_elf_32.cc
diff --git a/courgette/disassembler_elf_32.cc b/courgette/disassembler_elf_32.cc
index ff2b9e03b611cc42717aa96a2e7525023696a242..d313ca08ce8c2f84caf2198d1e2fa1e46ab87b58 100644
--- a/courgette/disassembler_elf_32.cc
+++ b/courgette/disassembler_elf_32.cc
@@ -262,6 +262,9 @@ CheckBool DisassemblerElf32::ParseFile(AssemblyProgram* program) {
 
     const Elf32_Shdr *section_header = SectionHeader(section_id);
 
+    if (section_header->sh_type == SHT_NOBITS)

[dgarrett, 2014/10/20 18:15:44]

This seems like the right thing to do... I'm surprised the original code was
working.

Did I have no NOBITS sections in the files it worked with?

There was substantial testing using a wide variety of binaries, but I have no
memory of how common NOBITS sections are.

[Will Harris, 2014/10/20 19:01:44]

NOBITS are in almost every ELF file as the .bss is always NOBITS as it's just
going to be some memory that the dynamic linker will zero out with enough space
for the globals.

In all the files I can compile, the .bss is near the end of the file (typically
before the comment or debug section) but the next section is either at the same
file offset or higher, which means the bug won't trigger.

e.g. a file I compile just now has

[24] .data             PROGBITS        0804a00c 00100c 000008 00  WA  0   0  4
[25] .bss              NOBITS          0804a014 001014 000008 00  WA  0   0  4
[26] .comment          PROGBITS        00000000 001014 00002a 01  MS  0   0  1

The issue with the ARC files was that I think they were manually shifting the
.bss beyond the end of the file, perhaps they are using a custom linker script
for this?  There is more info in the bug on why this causes a segfault.  I can't
find the makefile for libdvm.so to check what's special about it.

+      continue;
+
     if (!ParseSimpleRegion(file_offset,
                            section_header->sh_offset,
                            program))
@@ -282,8 +285,6 @@ CheckBool DisassemblerElf32::ParseFile(AssemblyProgram* program) {
           return false;
         file_offset = section_header->sh_offset + section_header->sh_size;
         break;
-      case SHT_NOBITS:
-        // Fall through
       case SHT_INIT_ARRAY:
         // Fall through
       case SHT_FINI_ARRAY: