mentioned in: bitcoin#34436
"> ```shell
fuzz: util/overflow.h:64: auto CeilDiv(const Dividend, const Divisor) [Dividend = long, Divisor = int]: Assertion `dividend >= 0 && divisor > 0' failed.
Yeah, a few places (e.g. GetVirtualTransactionSize) were replaced originally in the PR, but turns out fuzzers sometimes create negative amounts for these - not sure if that's possible in reality so I've reverted that one.
Sorry for all the pushes, the extra validations in the new CeilDiv trigger a lot of these (invalid?) edge cases - so we can investigate and possibly migrate them in follow-up PRs instead." - l0rinc
mentioned in: bitcoin#34436
"> ```shell
Yeah, a few places (e.g.
GetVirtualTransactionSize) were replaced originally in the PR, but turns out fuzzers sometimes create negative amounts for these - not sure if that's possible in reality so I've reverted that one.Sorry for all the pushes, the extra validations in the new
CeilDivtrigger a lot of these (invalid?) edge cases - so we can investigate and possibly migrate them in follow-up PRs instead." - l0rinc