[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