[PATCH] D76543: [llvm-dwp] Fix a possible out of bound access.

Igor Kudrin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 30 02:07:50 PDT 2020


ikudrin added inline comments.


================
Comment at: llvm/tools/llvm-dwp/llvm-dwp.cpp:250-252
       auto &C = Entry.Contributions[Kind - DW_SECT_INFO];
       C.Offset += I->Offset;
       C.Length = I->Length;
----------------
jhenderson wrote:
> I've not really looked around this code, so this might not make sense, but is it also worth checking that we don't emit data for these sections?
`llvm-dwp` does not emit anything for unknown sections and the patch is not changing that.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D76543/new/

https://reviews.llvm.org/D76543





More information about the llvm-commits mailing list