fix BIP141 nested P2SH scriptSig byte representation #375

pull voisine wants to merge 1 commits into bitcoin:master from voisine:patch-4 changing 1 files +1 −1
  1. voisine commented at 3:44 PM on April 22, 2016: contributor

    The byte representation of "<0 <32-byte-hash>>" is "0x220020{32-byte-hash}"

    What was listed here would be the byte representation of "0 <32-byte-hash>". The text explains that there is only one item in scriptSig, so I'm guessing the byte representation is wrong. Also the corrected byte representation would produce the same sig/pubkey described in P2WSH after following the bip16 rules.

  2. Update bip-0141.mediawiki
    The byte representation of "<0 <32-byte-hash>>" is "0x220020{32-byte-hash}"
    
    What was listed here would be the byte representation of "0 <32-byte-hash>". The text explains that there is only one item in scriptSig, so I'm guessing the byte representation is wrong. Also the corrected byte representation would produce the same sig/pubkey described in P2WSH after following the bip16 rules.
    d72c1bfc71
  3. jl2012 commented at 4:26 PM on April 22, 2016: contributor

    Thanks. ACK

  4. voisine renamed this:
    Update bip-0141.mediawiki
    fix P2SH nested scriptSig byte representation
    on Apr 22, 2016
  5. voisine renamed this:
    fix P2SH nested scriptSig byte representation
    fix BIP141 nested P2SH scriptSig byte representation
    on Apr 22, 2016
  6. luke-jr merged this on Apr 23, 2016
  7. luke-jr closed this on Apr 23, 2016

Contributors

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-04-14 18:10 UTC

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