DragonX/dragonx
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 2 Wiki Activity

More error checking and use a CAmount instead of int, which hopefully fixes CommitTransaction() bug

Browse Source
This commit is contained in:
Duke Leto
2020-03-10 09:10:05 -04:00
parent 1beb2b1fa8
commit 968890c591
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/wallet/asyncrpcoperation_saplingconsolidation.cpp
View File

@@ -181,6 +181,7 @@ bool AsyncRPCOperation_saplingconsolidation::main_impl() {
// Add the actual consolidation tx
builder.AddSaplingOutput(extsk.expsk.ovk, addr, amountToSend - fConsolidationTxFee);
LogPrint("zrpcunsafe", "%s: Added consolidation output", getId(), addr);
// Add sietch zouts
int MIN_ZOUTS = 7;
@@ -191,8 +192,11 @@ bool AsyncRPCOperation_saplingconsolidation::main_impl() {
auto zaddr = DecodePaymentAddress(zdust);
if (IsValidPaymentAddress(zaddr)) {
auto sietchZoutput = boost::get<libzcash::SaplingPaymentAddress>(zaddr);
int amount=0;
CAmount amount=0;
builder.AddSaplingOutput(extsk.expsk.ovk, sietchZoutput, amount);
} else {
LogPrint("zrpcunsafe", "%s: Invalid payment address! Stopping.", getId(), zaddr);
break;
}
}
//CTransaction tx = builder.Build();