-
-
Notifications
You must be signed in to change notification settings - Fork 385
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
panic: environment must be declarative #2719
Comments
Hey @langston-barrett this fuzzer is interesting, do you have a guide or steps on how you’re using this with Boa? the reference error seems correct but the panic shouldn’t be there. |
Thanks for the interest @jasonwilliams! Good news is, it's really easy! The simplest set-up would be: tree-crasher-javascript corpus/ boa where The one I actually use is:
That limits the RAM and network access of the fuzzed processes and also finds panics in additions to segfaults. I usually use a corpus of files that Documentation is available here: https://langston-barrett.github.io/tree-crasher/ |
This is fixed, just needs to be added to our tests to avoid regressions. |
Found with tree-crasher on 0e01a74. Let me know if you still want more bug reports, I'll stop posting for now to avoid spamming you!
The text was updated successfully, but these errors were encountered: