BIP-346: correct TxFieldSelector index upper bound #2093

pull yyhrnk wants to merge 1 commits into bitcoin:master from yyhrnk:fix/txfs-index-bounds changing 1 files +1 −1
  1. yyhrnk commented at 5:49 pm on February 2, 2026: contributor

    Previously parse_inout_selection only rejected indices strictly greater than nb_items, allowing idx == nb_items to pass through and later cause a panic when indexing transaction inputs/outputs.

    Tightened the bounds check in parse_inout_selection to treat idx >= nb_items as out of bounds, enforcing the intended 0..nb_items-1 index range for TxFieldSelector individual mode.

  2. fix: correct TxFieldSelector index upper bound e3c4dede41
  3. murchandamus added the label Typo-to-be-collected on Feb 3, 2026
  4. murchandamus added the label Pending acceptance on Feb 3, 2026
  5. murchandamus commented at 0:45 am on February 3, 2026: contributor

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-10 17:10 UTC

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