[PATCH] D39185: [llvm-dwarfdump] - Fix array out of bounds access crash.

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added a comment.

In https://reviews.llvm.org/D39185#905246, @grimar wrote:

> In https://reviews.llvm.org/D39185#905240, @aprantl wrote:
>
> > Out of curiosity: does llvm-dwarfdump --verify fail on your testcase?
>
>
> Yes, it complains:


Awesome!

> Verifying .debug_abbrev...
>  Verifying .debug_info Unit Header Chain...
>  error: Units[1] - start offset: 0x00000020
>  note: The length for this unit is too large for the .debug_info provided.
>  note: The 16 bit unit header version is not valid.
>  note: The address size is unsupported.
>  Verifying .debug_info references...
>  Errors detected.
>  warning: DWARF compile unit extends beyond its bounds cu 0x00000000 at 0x0000002
>  1'
>  warning: DWARF compile unit extends beyond its bounds cu 0x00000000 at 0x0000002
>  1'

Uh.. that means we either have a bug in our DWARF backend or in the Verifier (probably the Verifier). That also needs to be fixed at some point :-)


https://reviews.llvm.org/D39185