BIP324: Fix features bitmask for decoding-case selection #1969

pull phrwlk wants to merge 1 commits into bitcoin:master from phrwlk:Boban changing 1 files +4 −4
  1. phrwlk commented at 9:52 am on September 16, 2025: none
    Replace (features & 4) with (features & 3) in the four decoding-case checks. The docstring specifies that the lower two bits (f & 3) select which of x1/x2/x3 are valid. Using (features & 4) limited the value to {0,4}, making cases 1 and 2 unreachable and bypassing proper gating when bit 4 (u >= p) was set. This aligns implementation with the documented behavior and ensures all four modes are exercised correctly.
  2. BIP324: Fix features bitmask for decoding-case selection e78b33205b


phrwlk


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: 2025-09-16 21:10 UTC

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