-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Bug: memory allocation of 9223372036854775807 bytes failed
#7378
Comments
hmm, I'm unable to reproduce this locally. couldn't locate where this overflow before allocating might have happened. could you perhaps rerun with |
I can't reproduce this error locally neither. I only face it in the CI, unfortunately. 3 hrs ago I pushed a commit that passed the tests (cf. pcaversaccio/createx@5811b5a). So there happened something in between... |
Let me try locally using |
Can you try rerunning the CI? I'm wondering if this is maybe a fuzz or invariant test allocating a crazy big type? |
Rerunning here: |
Looks to have completed successfully (no core dump), albeit with failing tests. Am I reading correctly? |
It looks like this indeed but maybe it just didn't get there due to an early revert due to the failing test (which has never failed until now; maybe time for an additional |
the error looks like a classic allocate capacity with underflow error, but I was unable to identify where |
Still running locally to see if I can reproduce. This so weird really. Merged my PR in the meantime and you know what, it fucking passed: https://github.com/pcaversaccio/createx/actions/runs/8252329302/job/22571561906. I HATE SOFTWARE :) |
What would you identify as the largest fuzz/invariant tests in the code base? Might be worth trying to narrow down if it's one of the generators We should probably log the fuzz seed so this is more easily reproducible |
https://github.com/pcaversaccio/createx/blob/main/foundry.toml#L21
|
I tried reproduce the bug locally using these params but no luck. Maybe it's just a crazy weird edge case tbh, idk. If you think it's too hard to figure out the issue with the limited data we have, I can close it and reopen again later if I face it again? Thoughts? |
sg! |
No need since we have #7336 :) just make sure you save the failure file somewhere in CI and download locally to replay (if that's a fuzzed test failing) |
@grandizzy does this work even if forge ooms? |
obviously it doesn't :D nm |
Just faced another overflow bug here: #7394 |
Component
Forge
Have you ensured that all of these are up to date?
What version of Foundry are you on?
forge 0.2.0 (7545c7a 2024-03-12T00:16:54.294788023Z)
What command(s) is the bug in?
forge test
Operating System
Linux
Describe the bug
See CI run here:
I haven't changed the tests for 3 months, except for that PR pcaversaccio/createx#90 which raises now with that error.
The text was updated successfully, but these errors were encountered: