Skip to content

fix: Enforce recipients of l2l1msgs having 20 bytes at most#19898

Merged
sirasistant merged 9 commits intomerge-train/avmfrom
arv/fix_recipient_completeness_issue
Jan 26, 2026
Merged

fix: Enforce recipients of l2l1msgs having 20 bytes at most#19898
sirasistant merged 9 commits intomerge-train/avmfrom
arv/fix_recipient_completeness_issue

Conversation

@sirasistant
Copy link
Copy Markdown
Contributor

@sirasistant sirasistant changed the title fix: Enforce recipients of l2l1msgs having 20 bytes at most fix: Enforce recipients of l2l1msgs having 20 bytes at most (WIP) Jan 23, 2026
Comment thread barretenberg/cpp/pil/vm2/opcodes/send_l2_to_l1_msg.pil Outdated

namespace {

EthAddress ff_to_eth_address(FF ff)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

const&FF

Copy link
Copy Markdown
Contributor

@LeilaWang LeilaWang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only a small nit. Circuit side LGTM 😄

Comment thread noir-projects/noir-protocol-circuits/crates/types/src/constants.nr Outdated
Comment thread barretenberg/cpp/pil/vm2/opcodes/send_l2_to_l1_msg.pil Outdated
@sirasistant sirasistant changed the title fix: Enforce recipients of l2l1msgs having 20 bytes at most (WIP) fix: Enforce recipients of l2l1msgs having 20 bytes at most Jan 26, 2026
Copy link
Copy Markdown
Contributor

@IlyasRidhuan IlyasRidhuan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PIL LGTM!

Copy link
Copy Markdown
Contributor

@jeanmon jeanmon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Please consider my comments.

Comment thread barretenberg/cpp/pil/vm2/opcodes/send_l2_to_l1_msg.pil Outdated
@sirasistant sirasistant merged commit 2476d25 into merge-train/avm Jan 26, 2026
8 checks passed
@sirasistant sirasistant deleted the arv/fix_recipient_completeness_issue branch January 26, 2026 12:07
@AztecBot AztecBot mentioned this pull request Jan 26, 2026
github-merge-queue Bot pushed a commit that referenced this pull request Jan 26, 2026
BEGIN_COMMIT_OVERRIDE
fix: SLOAD fuzzing after configurable addr (#19895)
fix(avm): Make the TS simulator lazy read calldata / return data
(#19843)
chore(avm): ECC pre-audit gadget fuzzing (#19739)
fix: Fuzzer sometimes creates to radix with num_limbs=0 (#19929)
fix: Enforce recipients of l2l1msgs having 20 bytes at most (#19898)
fix merge conflict
END_COMMIT_OVERRIDE
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.

5 participants