Skip to content

Commit ffa5159

Browse files
committed
Merge #11508: Fix crash via division by zero assertion
207408b Fix crash via division by zero assertion (Jonas Schnelli) Pull request description: Replaces the newly added `assert` for a devision by zero protection by a control structure. Floating point division by zero is defined by the floating point standard and results in +inf or -inf. Introduced in #11133 Reported by @mzhou, fixes #11501 Tree-SHA512: ac9b4efa3ba52a2aa246fb11170128c4aaf829fd491b649524c85069c6ed33ae612e761809aea9d9a44bdea29a417b3f3a558226495094b5070a42a56b2ac77e
2 parents 26fee4f + 207408b commit ffa5159

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

src/qt/coincontroldialog.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -582,8 +582,7 @@ void CoinControlDialog::updateLabels(WalletModel *model, QDialog* dialog)
582582
QString toolTipDust = tr("This label turns red if any recipient receives an amount smaller than the current dust threshold.");
583583

584584
// how many satoshis the estimated fee can vary per byte we guess wrong
585-
assert(nBytes != 0);
586-
double dFeeVary = (double)nPayFee / nBytes;
585+
double dFeeVary = (nBytes != 0) ? (double)nPayFee / nBytes : 0;
587586

588587
QString toolTip4 = tr("Can vary +/- %1 satoshi(s) per input.").arg(dFeeVary);
589588

0 commit comments

Comments
 (0)