fix: swap scriptPathControlBlocks[1] and [2] for three-leaf P2MR test vectors #2102

pull jasonandjay wants to merge 1 commits into bitcoin:master from jasonandjay:fix/swap-scriptpath-control-blocks-three-leaf changing 1 files +4 −4
  1. jasonandjay commented at 1:57 am on February 15, 2026: none

    Summary

    Problem

    In a tree structured as [leaf0, [leaf1, leaf2]], the control block for spending a leaf must contain the sibling hashes along the Merkle path, not the leaf’s own hash. The current test vectors have the control blocks for leaf1 and leaf2 swapped, causing them to include the spending leaf’s own hash as the first path element instead of its sibling’s hash.

    Example:es 1 and 2 in the

    three-lea Intermediate leaf hashes (correct in the test vector):

    0leafHash[0] = 2645a02e0aac1fe69d69755733a9b7621b694bb5b5cde2bbfc94066ed62b9817
    1leafHash[1] = ba982a91d4fc552163cb1c0da03676102d5b7a014304c01f0c77b2b8e888de1c
    2leafHash[2] = 9e31407bffa15fefbf5090b149d53959ecdf3f62b1246780238c24501d5ceaf6
    3merkleRoot  = ccbd66c6f7e8fdab47b3a486f59d28262be857f30d4773f2d5ea47f7761ce0e2
    4Current (incorrect)ests/data/p2mr_construction. 
    5[1] = c1 ba982a91...de1c 2645a02e...9817    first path element is leafHash[1] itself!
    6[2] = c1 9e31407b...f6   2645a02e...9817    first path element is leafHash[2] itself!
    

    Fix swapped

    0scripto verify leaf1:kj = leafHash[1] = ba982a91...
    1ej = ba982a91...  (from control block, which IS leafHash[1])
    2→ tapBranchHash(ba982a91..., ba982a91...) = ccabeef4...  ← hashing with itself!
    3→ final root = 5037e24f... ≠ expected ccbd66c6...  ✗
    

    Fix

    Swap1 and 2 in the three-leaf P2MRand scriptPathControlBlocks[2]:

    0[1] = c1 9e31407b...f6   2645a02e...9817   ← sibling of leaf1 is leafHash[2], then leafHash[0]
    1[2] = c1 ba982a91...de1c 2645a02e...9817   ← sibling of leaf2 is leafHash[1], then leafHash[0]
    

    Verification of correctedontrolBlocks

    0entries for leaffor leaf1:kj = leafHash[1] = ba982a91...  
    1ej = leafHash[2] = 9e31407b...  
    2→ tapBranchHash(9e31..., ba98...) = ffe578e9...  ← correct inner branch hash ✓. 
    3ej = leafHash[0] = 2645a02e...  
    4→ tapBranchHash(2645..., ffe5...) = ccbd66c6...  ← correct merkle root ✓
    

    The same issue exists in P2MR test vectors p2mr_threeand is fixed identically.

  2. fix: swap scriptPathControlBlocks[1] and [2] for three-leaf P2MR test vectors
    In a tree structured as [leaf0, [leaf1, leaf2]], the control block for
    spending a leaf must contain the sibling hashes along the Merkle path,
    not the leaf's own hash. The current test vectors have the control blocks
    for leaf1 and leaf2 swapped, causing them to include the spending leaf's
    own hash as the first path element instead of its sibling's hash.
    
    Fix swapped scriptPathControlBlocks entries for:
    - p2mr_three_leaf_complex
    - p2mr_three_leaf_alternative
    3081e08900


jasonandjay


github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-02-17 00:10 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me