search

CodeIntelligenceTesting / jazzer

Coverage-guided, in-process fuzzing for the JVM
https://code-intelligence.com
Other
1.03k stars 137 forks source link

Fix Clojure hooks #827

Closed fmeum closed 1 year ago

fmeum commented 1 year ago

@zgtm Could you check whether the other hooks are affected by this problem as well?

fmeum commented 1 year ago

@zgtm I reduced the variance in test runtime by adding -use_value_profile=1. CI is green now.

zgtm commented 1 year ago

@zgtm Could you check whether the other hooks are affected by this problem as well?

This should also affect the hook numberUnsignedDivide in src/main/java/com/code_intelligence/jazzer/runtime/TraceDivHooks.java. Though I have not been able to tweak the test to show different behaviour, here …

fmeum commented 1 year ago

Thanks for finding and fixing this! I've added another commit, feel free to squash into your first commit!

Made the exact same change at a similar time (only with an additional comment) :-)