Skip to content

Commit 03403d8

Browse files
committed
Merge pull request #6951
513686d [qt] Use maxTxFee instead of 10000000 (MarcoFalke)
2 parents 7f8e90d + 513686d commit 03403d8

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/qt/sendcoinsdialog.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -531,7 +531,7 @@ void SendCoinsDialog::processSendCoinsReturn(const WalletModel::SendCoinsReturn
531531
msgParams.second = CClientUIInterface::MSG_ERROR;
532532
break;
533533
case WalletModel::AbsurdFee:
534-
msgParams.first = tr("A fee higher than %1 is considered an absurdly high fee.").arg(BitcoinUnits::formatWithUnit(model->getOptionsModel()->getDisplayUnit(), 10000000));
534+
msgParams.first = tr("A fee higher than %1 is considered an absurdly high fee.").arg(BitcoinUnits::formatWithUnit(model->getOptionsModel()->getDisplayUnit(), maxTxFee));
535535
break;
536536
case WalletModel::PaymentRequestExpired:
537537
msgParams.first = tr("Payment request expired.");

src/qt/walletmodel.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -290,8 +290,10 @@ WalletModel::SendCoinsReturn WalletModel::prepareTransaction(WalletModelTransact
290290
return TransactionCreationFailed;
291291
}
292292

293-
// reject absurdly high fee > 0.1 bitcoin
294-
if (nFeeRequired > 10000000)
293+
// reject absurdly high fee. (This can never happen because the
294+
// wallet caps the fee at maxTxFee. This merely serves as a
295+
// belt-and-suspenders check)
296+
if (nFeeRequired > maxTxFee)
295297
return AbsurdFee;
296298
}
297299

0 commit comments

Comments
 (0)