Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge bitcoin#29462: [fuzz] Avoid partial negative result
9dae3b9 [fuzz] Avoid partial negative result (Murch) Pull request description: May address the problem reported by maflcko in bitcoin#27877 (review). 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 9dae3b9 sipa: utACK 9dae3b9 achow101: ACK 9dae3b9 brunoerg: crACK 9dae3b9 Tree-SHA512: 744b4706268d8dfd77538b99492ecf3cf77d229095f9bcd416a412131336830e2f134f2b2846c79abd3d193426f97c1f71eeaf68b16ab00e76318d57ee3673c7
- Loading branch information