Skip to content

fix: Restore signer nonce in NestedMultisigBuilder#79

Merged
mdehoog merged 1 commit intobase:mainfrom
clabby:cl/fix-nested-safe-nonce
Apr 26, 2024
Merged

fix: Restore signer nonce in NestedMultisigBuilder#79
mdehoog merged 1 commit intobase:mainfrom
clabby:cl/fix-nested-safe-nonce

Conversation

@clabby
Copy link
Contributor

@clabby clabby commented Apr 25, 2024

Overview

Fixes a bug in the NestedMultisigBuilder, where the signer nonce was not restored during the signing process. This caused eip712 signer to sign the wrong message.

@cb-heimdall
Copy link
Collaborator

Review Error for mds1 @ 2024-04-25 22:17:15 UTC
User must have write permissions to review

mds1 added a commit to ethereum-optimism/superchain-ops that referenced this pull request Apr 26, 2024
@mdehoog mdehoog merged commit 323c957 into base:main Apr 26, 2024
github-merge-queue bot pushed a commit to ethereum-optimism/superchain-ops that referenced this pull request Apr 26, 2024
joshH94 added a commit to joshH94/superchain-ops that referenced this pull request Sep 22, 2025
radiant-lab7098 added a commit to radiant-lab7098/superchain-ops that referenced this pull request Sep 29, 2025
async-druidxbh6 added a commit to async-druidxbh6/superchain-ops that referenced this pull request Nov 13, 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.

4 participants