[PATCH] D62226: [libFuzzer] Ignore synthetic exceptions on Fuchsia

Roland McGrath via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 21 18:36:35 PDT 2019


mcgrathr accepted this revision.
mcgrathr added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: dexonsmith.

lgtm with one nit: either change to an assert or improve the comment to clarify what other cases are expected



================
Comment at: compiler-rt/lib/fuzzer/FuzzerUtilFuchsia.cpp:244
+    // Ignore signals and informational synthetic exceptions.
+    if (!ZX_PKT_IS_EXCEPTION(Packet.type) ||
+        ZX_EXCP_THREAD_STARTING == Packet.type ||
----------------
I think !ZX_PKT_IS_EXCEPTION here is unexpected and would indicate a bug.
There should be no packets received except the ones you've asked for.
Probably you should just assert that it is.


Repository:
  rCRT Compiler Runtime

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

https://reviews.llvm.org/D62226





More information about the llvm-commits mailing list