Skip to content

Fix/flounder audit circuit#416

Open
blake-pro wants to merge 3 commits into
GOATNetwork:gc-v2from
blake-pro:fix/flounder-audit-circuit
Open

Fix/flounder audit circuit#416
blake-pro wants to merge 3 commits into
GOATNetwork:gc-v2from
blake-pro:fix/flounder-audit-circuit

Conversation

@blake-pro

Copy link
Copy Markdown
Contributor

This PR fixed the following issues:

  1. Bind recursive header, commit, and state chain states to the verified zkm_public_values, removing the prover-controlled state payload from PrevProof. It also propagates host-side proof verification failures as errors instead of panicking.
  2. Bind the commit-chain output transaction to the transaction proven by SPV, preventing an unrelated valid SPV proof from substituting for the target commit.
  3. Prevent tampered Groth16 vk and unsafe archive paths from compromising the node.

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.

1 participant