Re: [PATCH 1/1] gdb: Fix segfault with a negative .dynamic section size

[Tom Tromey <tom@tromey.com>]

>>>>> Willgerodt, Felix <felix.willgerodt@intel.com> writes:

> BFD is checking the section size against the file size two times and once
> It checks if the section size is staying in the ELF segments region.
> I don't think another warning from GDB would help.  Or what GDB at this
> part of the code could do differently if it encounters such a thing.

Ok, I understand the problem now, I think.

bfd_get_section_contents does the size check -- but the buffer for the
contents must be allocated before this call.  So, this approach is
forced on us by BFD's API.

Maybe changing that would be good, but meh, (1) changing BFD can be
difficult, and (2) it seems like a lot of work for fuzzer input.

I do wonder now why gdb generally uses bfd_get_section_contents and not
bfd_get_full_section_contents, as the latter supports decompression while 
former does not.

Tom