Skip to content

Reports batch - #52

Merged
davxy merged 13 commits into
mainfrom
reports-batch
Sep 3, 2025
Merged

davxy merged 13 commits into
mainfrom
reports-batch

Conversation

@davxy

@davxy davxy commented Sep 2, 2025

Copy link
Copy Markdown
Owner

No description provided.

@dakk dakk mentioned this pull request Sep 2, 2025
@clearloop

Copy link
Copy Markdown
Contributor

interesting tests, hit my recent optimizations lmao

@clearloop clearloop mentioned this pull request Sep 2, 2025
1 task
@dakk

dakk commented Sep 2, 2025

Copy link
Copy Markdown

The table shows that jampy is failing at 1756792661, but I can't find the report in fuzz-reports/0.7.0/reports/jampy/

@sourabhniyogi

sourabhniyogi commented Sep 3, 2025 •

Copy link
Copy Markdown
Contributor

For 1756792661, it appears this is the first case where we have invalid state transition (requiring that we respond with a post-state-root being the same as the pre-state-root) -- in this case, we believe, due to an invalid guarantee signature in the first state transition (00000027.bin) making it invalid -- and where we consider the second state transition valid. Are we misunderstanding what is going on or is there something incorrect about how our fuzzer target is dealing with the first invalid state transition?

@vekexasia

Copy link
Copy Markdown
Contributor

For 1756792661, it appears this is the first case where we have invalid state transition (requiring that we respond with a post-state-root being the same as the pre-state-root) -- in this case, we believe, due to an invalid guarantee signature in the first state transition (00000027.bin) making it invalid -- and where we consider the second state transition valid. Are we misunderstanding what is going on or is there something incorrect about how our fuzzer target is dealing with the first invalid state transition?

My fuzzer had issues with it as well. but target handled it fine.

If you're replciating the traces yourself like I am doing, i now set pre_state as cur, state, try to apply 27 (failure - state remains the old one), then try to apply 28 (ok)

@davxy
davxy merged commit fce84b2 into main Sep 3, 2025
@davxy
davxy deleted the reports-batch branch September 3, 2025 06:36
@sourabhniyogi

Copy link
Copy Markdown
Contributor

For 1756792661, it appears this is the first case where we have invalid state transition (requiring that we respond with a post-state-root being the same as the pre-state-root) -- in this case, we believe, due to an invalid guarantee signature in the first state transition (00000027.bin) making it invalid -- and where we consider the second state transition valid. Are we misunderstanding what is going on or is there something incorrect about how our fuzzer target is dealing with the first invalid state transition?

My fuzzer had issues with it as well. but target handled it fine.

If you're replciating the traces yourself like I am doing, i now set pre_state as cur, state, try to apply 27 (failure - state remains the old one), then try to apply 28 (ok)

Ok will see how to get it to behave and implement v1 features at the same time, thank you!

0xjunha added a commit to fastroll-jam/fastroll that referenced this pull request Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants