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

BIP-322 Proof of Reserves

Public commit record

What the developer wrote

Authored by scgbckbone

45/100 · Thin
BIP-322 Proof of Reserves
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
The short version

What changed, and why it matters

This commit adds support for BIP-322 Proof of Reserves signing on COLDCARD hardware wallets. It lets users prove they control certain Bitcoin addresses by signing a special non-spending transaction, without actually moving funds. The change also allows transaction version 0 only for these proof-of-reserve transactions, and blocks it for normal payments to prevent potential misuse.

Recommended action

Review the POR detection heuristic carefully: ensure a malformed PSBT cannot trigger POR mode and bypass fee/consolidation checks. Verify that nVersion=0 cannot be accepted for any non-POR transaction, and that the 'to_spend' deserialization and prevout/scriptSig assertions are robust against malicious UTXO data. Run the new test suite, especially the negative tests for invalid to_spend fields and nVersion zero normal transactions.

Security signals we found

01

New transaction parsing path accepts nVersion=0 under a specific POR heuristic

02

POR detection relies on a single zero-amount OP_RETURN output (scriptPubKey == b'\x6a')

03

POR validation parses and constrains the embedded 'to_spend' transaction from the 0th input's non-witness UTXO

04

Normal fee and amount checks are bypassed for POR transactions

05

Sighash restricted to SIGHASH_ALL for POR inputs

06

Foreign inputs are prohibited for POR

07

UI title changed from 'OK TO SEND?' to 'OK TO SIGN?' for POR to reduce payment-confusion risk

Risk score

Why this scored 37/100

Our methodology →
Potential impact 8/30
Exploitability 6/25
Stealth signal 5/15
Affected reach 7/15
Confidence 7/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.