Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add simulator error when reentrancy occurs #1653

Closed
Tracked by #1601
dbanks12 opened this issue Aug 18, 2023 · 1 comment
Closed
Tracked by #1601

Add simulator error when reentrancy occurs #1653

dbanks12 opened this issue Aug 18, 2023 · 1 comment
Milestone
🚀 Initial F&F San...

Comments

@dbanks12
Copy link
Collaborator

No description provided.

@dbanks12 dbanks12 added this to A3 Aug 18, 2023
@github-project-automation github-project-automation bot moved this to Todo in A3 Aug 18, 2023
@dbanks12 dbanks12 mentioned this issue Aug 18, 2023
Closed
@dbanks12
Copy link
Collaborator Author

"I guess when I wrote that yesterday, I was misremembering the actual problem we had. This pattern could cause errors if we don't correctly order (in the simulator) state updates and the ordering of enqueueing public functions." - @iAmMichaelConnor

@dbanks12 dbanks12 closed this as not planned Won't fix, can't repro, duplicate, stale Aug 18, 2023
@github-project-automation github-project-automation bot moved this from Todo to Done in A3 Aug 18, 2023
codygunton pushed a commit that referenced this issue Jan 23, 2024
@guidovranken
Fully initialize members in client_proofs_join_split_tx test (#1653)
ac153f5 
Fixes https://github.com/AztecProtocol/aztec2-internal/issues/1652
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
A3
Archived in project
Milestone
🚀 Initial F&F Sandbox Release
Development

No branches or pull requests
1 participant
@dbanks12