Missing copy constraint in halo2_gadgets variable-base scalar multiplication allows under-constrained base, breaking Orchard Action circuit soundness
Description
ZEBRA is a Zcash node written entirely in Rust. Prior to zebrad 5.0.0, halo2_gadgets 0.5.0, orchard 0.14.0, zcash_primitives 0.28.0, and zcashd 6.20.0, the variable-base scalar multiplication gadget in halo2_gadgets/src/ecc/chip/mul/incomplete.rs used assign_advice() for the base point without a copy constraint tying it to the actual base, allowing a malicious prover to produce a valid proof for an Orchard Action with an under-constrained base point and bypass the diversified-address-integrity check that binds pk_d, g_d, ivk, the nullifier (nf), and the spend validating key (ak) to the note being spent. This issue is fixed in zebrad 5.0.0, halo2_gadgets 0.5.0, orchard 0.14.0, zcash_primitives 0.28.0, and zcashd 6.20.0.
Affected packages
Versions sourced from the GitHub Security Advisory.
| Package | Affected versions | Patched versions |
|---|---|---|
zebradcrates.io | < 5.0.0 | 5.0.0 |
halo2_gadgetscrates.io | < 0.5.0 | 0.5.0 |
orchardcrates.io | < 0.14.0 | 0.14.0 |
zcash_primitivescrates.io | < 0.28.0 | 0.28.0 |
Affected products
4v4.5.1, v4.5.0, v4.4.0, …+ 1 more
- (no CPE)range: v4.5.1, v4.5.0, v4.4.0, …
- (no CPE)range: <5.0.0
- Range: <0.5.0
- Range: <0.14.0
Patches
Vulnerability mechanics
References
8- github.com/advisories/GHSA-ww9q-8r59-xv46ghsaADVISORY
- github.com/ZcashFoundation/zebra/releases/tag/v5.0.0mitrex_refsource_MISC
- github.com/ZcashFoundation/zebra/security/advisories/GHSA-ww9q-8r59-xv46ghsax_refsource_CONFIRMWEB
- github.com/zcash/halo2/releases/tag/halo2_gadgets-0.5.0mitrex_refsource_MISC
- github.com/zcash/librustzcash/releases/tag/zcash_primitives-0.28.0mitrex_refsource_MISC
- github.com/zcash/orchard/releases/tag/0.14.0mitrex_refsource_MISC
- github.com/zcash/zcash/releases/tag/v6.20.0mitrex_refsource_MISC
- zfnd.org/zebra-4-5-3-and-5-0-0-emergency-soft-fork-and-nu6-2-activationmitrex_refsource_MISC
News mentions
0No linked articles in our index yet.