diff options
| author | Ava Chow <github@achow101.com> | 2024-02-22 08:59:12 -0500 |
|---|---|---|
| committer | Ava Chow <github@achow101.com> | 2024-02-22 08:59:30 -0500 |
| commit | 1ac627c485a43e50a9a49baddce186ee3ad4daad (patch) | |
| tree | 73ed15fa6d987b4f0565229e9719e4e595822791 /src/wallet/scriptpubkeyman.h | |
| parent | 88b1229c134fa006d9a57e908ebebec944419347 (diff) | |
| parent | 9dae3b970a7a82e8d9f3f755048d427da78c49da (diff) | |
| download | bitcoin-1ac627c485a43e50a9a49baddce186ee3ad4daad.tar.xz | |
Merge bitcoin/bitcoin#29462: [fuzz] Avoid partial negative result
9dae3b970a7a82e8d9f3f755048d427da78c49da [fuzz] Avoid partial negative result (Murch)
Pull request description:
May address the problem reported by maflcko in https://github.com/bitcoin/bitcoin/pull/27877#pullrequestreview-1890304914.
For some values, `MAX_MONEY - max_spendable - max_output_groups` could result in a partial negative value. By putting the addition of `group_pos.size()` first, all partial results in this line will be strictly positive.
I opened this as a draft, since I was unable to reproduce the issue, so I’m waiting for confirmation whether this in fact mitigates the problem.
ACKs for top commit:
maflcko:
ACK 9dae3b970a7a82e8d9f3f755048d427da78c49da
sipa:
utACK 9dae3b970a7a82e8d9f3f755048d427da78c49da
achow101:
ACK 9dae3b970a7a82e8d9f3f755048d427da78c49da
brunoerg:
crACK 9dae3b970a7a82e8d9f3f755048d427da78c49da
Tree-SHA512: 744b4706268d8dfd77538b99492ecf3cf77d229095f9bcd416a412131336830e2f134f2b2846c79abd3d193426f97c1f71eeaf68b16ab00e76318d57ee3673c7
Diffstat (limited to 'src/wallet/scriptpubkeyman.h')
0 files changed, 0 insertions, 0 deletions