[PATCH] D38512: Added phdr upper bound checks to ElfObject

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 4 13:35:04 PDT 2017


Parker Thompson via Phabricator <reviews at reviews.llvm.org> writes:

> mothran added a comment.
>
> I am unable to construct a correctly malformed file with yaml2obj, but was able to create a test with hexedit.
> This is my first time committing a patch to LLVM and was wondering, do I have to use arc to submit a patch with a binary file?  And should I do that as a follow up commit or a whole new review and one patch?

Using phabricator is optional. Feel free to just attach a patch with the
binary test if you want.

Cheers,
Rafael


More information about the llvm-commits mailing list