fix invalid memory access that I added when merging descriptor wallets

Thanks, ubsan + fuzzer!
ElementsProject · Mar 26, 2021 · fec99ef · fec99ef
1 parent 98d0ff7
commit fec99ef
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/src/script/sign.cpp b/src/script/sign.cpp
@@ -506,11 +506,11 @@ bool SignTransaction(CMutableTransaction& mtx, const SigningProvider* keystore,
             prevTxOut = GetPeginOutputFromWitness(mtx.witness.vtxinwit[i].m_pegin_witness);
         } else {
             auto coin = coins.find(txin.prevout);
-            prevTxOut = coin->second.out;
             if (coin == coins.end() || coin->second.IsSpent()) {
                 input_errors[i] = "Input not found or already spent";
                 continue;
             }
+            prevTxOut = coin->second.out;
         }
 
         const CScript& prevPubKey = prevTxOut.scriptPubKey;