fuzz: Use CAmount for storing best_waste #34939

1. 
   achow101 commented at 0:32 am on March 28, 2026: member

   Waste is a CAmount, which is an int64_t. This will overflow an int, so best_waste should also be a CAmount.

   Fixes #34936

2. fuzz: Use CAmount for storing best_waste

   Waste is a CAmount, which is an int64_t. This will overflow an int, so
   `best_waste` should also be a `CAmount`.

   890a09b1e4
3. DrahtBot added the label Fuzzing on Mar 28, 2026
4. 
   DrahtBot commented at 0:32 am on March 28, 2026: contributor

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Reviews

   See the guideline for information on the review process.

   Type	Reviewers
   ACK	murchandamus, furszy

   If your review is incorrectly listed, please copy-paste <!–meta-tag:bot-skip–> into the comment that the bot should ignore.

5. 
   murchandamus commented at 1:06 am on March 28, 2026: member

   Tested againts the fuzz input provided in #32894 / #34936. Thanks Ava!

   ACK 890a09b1e49925315a5636f29cc5af36928fe092

6. 
   furszy commented at 2:54 am on March 28, 2026: member

   ACK 890a09b1e49925315a5636f29cc5af36928fe092
7. fanquake added this to the milestone 31.0 on Mar 28, 2026
8. fanquake added the label Needs Backport (31.x) on Mar 28, 2026
9. 
   yancyribbens commented at 2:02 pm on March 28, 2026: contributor

   best waste should be 2100000000000000 at most if in sats, right? Makes sense that this is larger than int max of 2,147,483,647.

Contributors
achow101 DrahtBot murchandamus furszy yancyribbens

Labels
Fuzzing Needs Backport (31.x)

Milestone
31.0