Merge #9302: Return txid even if ATMP fails for new transaction
b3a7410
Return txid even if ATMP fails for new transaction (Pieter Wuille)
This commit is contained in:
commit
b6abdc77d3
1 changed files with 4 additions and 4 deletions
|
@ -2584,11 +2584,11 @@ bool CWallet::CommitTransaction(CWalletTx& wtxNew, CReserveKey& reservekey, CCon
|
|||
{
|
||||
// Broadcast
|
||||
if (!wtxNew.AcceptToMemoryPool(maxTxFee, state)) {
|
||||
// This must not fail. The transaction has already been signed and recorded.
|
||||
LogPrintf("CommitTransaction(): Error: Transaction not valid, %s\n", state.GetRejectReason());
|
||||
return false;
|
||||
LogPrintf("CommitTransaction(): Transaction cannot be broadcast immediately, %s\n", state.GetRejectReason());
|
||||
// TODO: if we expect the failure to be long term or permanent, instead delete wtx from the wallet and return failure.
|
||||
} else {
|
||||
wtxNew.RelayWalletTransaction(connman);
|
||||
}
|
||||
wtxNew.RelayWalletTransaction(connman);
|
||||
}
|
||||
}
|
||||
return true;
|
||||
|
|
Loading…
Reference in a new issue