AI-generated analysisPublished automatically and not human-verified. Validated context appears in community notes below.
← Watch feed
High 71 Bitcoin

Reject duplicate Orchard and Ironwood action rk values

Public commit record

What the developer wrote

Authored by Adam Tucker

50/100 · Thin
Reject duplicate Orchard and Ironwood action rk values
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
The short version

What changed, and why it matters

This update fixes a flaw in how the Keystone hardware wallet checks Zcash private transaction proposals (PCZTs). If the same random verification key ('rk') appeared in two different spending actions—either two Orchard actions or one Orchard and one Ironwood action—the device would not notice. Because both action types sign the same overall transaction digest, a valid signature for one action could be copied and reused for the other, potentially letting an attacker authorize a spend they should not. The fix makes the wallet reject any PCZT that reuses such a key across actions.

Recommended action

Treat this as a security-hardening fix and include it in the next firmware release. Review whether any other shielded protocols or signature domains in the codebase share the same sighash and could have analogous duplicate-key replay issues. Ensure the new validation runs on all PCZT entry points, including any future batch or cypherpunk variants.

Security signals we found

01

New validation rejects duplicate Orchard/Ironwood randomized validating keys (rk)

02

Code comment explicitly describes signature replay risk across shielded actions

03

Tests added for duplicate rk within Orchard and across Orchard/Ironwood

04

All parse/check/sign code paths tested for rejection

05

CHANGELOG describes the change as a bug fix for Zcash PCZT rk reuse

Risk score

Why this scored 71/100

Our methodology →
Potential impact 22/30
Exploitability 16/25
Stealth signal 11/15
Affected reach 10/15
Confidence 8/10
Evidence quality 4/5
Human-validated context

Community notes

Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.

No validated notes yet.

The AI analysis stands alone for now. Submit a note if you can add evidence or important context.