This is an archive of the discontinued LLVM Phabricator instance.

Add fuzzer for Rust demangler
ClosedPublic

Authored by tmiasko on May 4 2021, 2:13 AM.

Diff Detail

Event Timeline

tmiasko created this revision.May 4 2021, 2:13 AM
tmiasko requested review of this revision.May 4 2021, 2:13 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 4 2021, 2:13 AM
dblaikie accepted this revision.May 4 2021, 2:57 PM

Looks good, thanks!

llvm/tools/llvm-rust-demangle-fuzzer/llvm-rust-demangle-fuzzer.cpp
17–19

Since free(nullptr) is a no-op, you could skip the 'if' here if you like.

This revision is now accepted and ready to land.May 4 2021, 2:57 PM
tmiasko updated this revision to Diff 343050.May 5 2021, 8:03 AM

Remove unnecessary nullptr check before free.

Could you commit this for me? Thank you.

Tomasz Miąsko <tomasz.miasko@gmail.com>

This revision was landed with ongoing or failed builds.May 5 2021, 12:51 PM
Closed by commit rG0e7c2aeaa8c0: Add fuzzer for Rust demangler (authored by tmiasko, committed by dblaikie). · Explain Why
This revision was automatically updated to reflect the committed changes.