MozillaSecurity / funfuzz

A collection of fuzzers in a harness for testing the SpiderMonkey JavaScript engine.
Mozilla Public License 2.0
630 stars 115 forks source link

[jsfunfuzz] Make testMathyFunction call itself or turn it recursive #91

Open nth10sd opened 7 years ago

nth10sd commented 7 years ago

testMathyFunction:

Potential improvement:

Sidenote:

twahlfeld commented 7 years ago

I have made testMathyFunction recursive. While profiling we noticed that for each average case loop went from 10s to a range of 10-100 (our timeout). However within minutes we find bugs related to 1357149. Is it worth the dramatic increase and variation in each test case?

nth10sd commented 7 years ago

Hmmm, I don't think so, especially since unhandlable ooms like bug 1357149 might be intended. (Let's wait for Jon Coppeard to comment there first). This was an idea, not sure how practical it is. Might you create a PR and we'll see what happens next?