GlcoinCore/elements
1
0
Fork 0
mirror of https://github.com/ElementsProject/elements.git synced 2026-08-16 13:01:19 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix invalid memory access that I added when merging descriptor wallets

Browse source 
Thanks, ubsan + fuzzer!
This commit is contained in:
Andrew Poelstra 2020-12-15 00:38:34 +00:00
parent 98d0ff7035
commit fec99efb8f
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/script/sign.cpp
View file

@ -506,11 +506,11 @@ bool SignTransaction(CMutableTransaction& mtx, const SigningProvider* keystore,
prevTxOut = GetPeginOutputFromWitness(mtx.witness.vtxinwit[i].m_pegin_witness); prevTxOut = GetPeginOutputFromWitness(mtx.witness.vtxinwit[i].m_pegin_witness);
} else { } else {
auto coin = coins.find(txin.prevout); auto coin = coins.find(txin.prevout);
prevTxOut = coin->second.out;
if (coin == coins.end() || coin->second.IsSpent()) { if (coin == coins.end() || coin->second.IsSpent()) {
input_errors[i] = "Input not found or already spent"; input_errors[i] = "Input not found or already spent";
continue; continue;
} }
prevTxOut = coin->second.out;
} }
const CScript& prevPubKey = prevTxOut.scriptPubKey; const CScript& prevPubKey = prevTxOut.scriptPubKey;