diff options
author | Gregory Sanders <gsanders87@gmail.com> | 2019-12-19 10:32:44 -0500 |
---|---|---|
committer | Gregory Sanders <gsanders87@gmail.com> | 2019-12-19 11:44:41 -0500 |
commit | 6020ce3c01fe5ee15a236c47da23342005b63055 (patch) | |
tree | 2db94d76e38531485c1f066342527736180a980e | |
parent | 988eaf2fcb71284c5fc57c7ef755bdafa4b929a0 (diff) | |
download | bitcoin-6020ce3c01fe5ee15a236c47da23342005b63055.tar.xz |
DecodeHexTx: Try case where txn has inputs first
-rw-r--r-- | src/core_read.cpp | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/src/core_read.cpp b/src/core_read.cpp index a3c9cf0159..da76983708 100644 --- a/src/core_read.cpp +++ b/src/core_read.cpp @@ -117,11 +117,12 @@ bool DecodeHexTx(CMutableTransaction& tx, const std::string& hex_tx, bool try_no std::vector<unsigned char> txData(ParseHex(hex_tx)); - if (try_no_witness) { - CDataStream ssData(txData, SER_NETWORK, PROTOCOL_VERSION | SERIALIZE_TRANSACTION_NO_WITNESS); + if (try_witness) { + CDataStream ssData(txData, SER_NETWORK, PROTOCOL_VERSION); try { ssData >> tx; - if (ssData.eof() && (!try_witness || CheckTxScriptsSanity(tx))) { + // If transaction looks sane, we don't try other mode even if requested + if (ssData.empty() && (!try_no_witness || CheckTxScriptsSanity(tx))) { return true; } } catch (const std::exception&) { @@ -129,8 +130,8 @@ bool DecodeHexTx(CMutableTransaction& tx, const std::string& hex_tx, bool try_no } } - if (try_witness) { - CDataStream ssData(txData, SER_NETWORK, PROTOCOL_VERSION); + if (try_no_witness) { + CDataStream ssData(txData, SER_NETWORK, PROTOCOL_VERSION | SERIALIZE_TRANSACTION_NO_WITNESS); try { ssData >> tx; if (ssData.empty()) { |