diff options
author | Suhas Daftuar <sdaftuar@gmail.com> | 2020-01-29 14:09:08 -0500 |
---|---|---|
committer | Suhas Daftuar <sdaftuar@gmail.com> | 2020-07-18 19:00:02 -0400 |
commit | 8e68fc246d09f1e6c6dfa8c676969d97c2eb4334 (patch) | |
tree | 799b4136a68960f75dc50ffce6c3ea6da908f273 /src/protocol.cpp | |
parent | 144c38582042c3b9ec53bb97ba0644fc0114f8fb (diff) | |
download | bitcoin-8e68fc246d09f1e6c6dfa8c676969d97c2eb4334.tar.xz |
Add wtxids to recentRejects instead of txids
Previously, we only added txids to recentRejects if we were sure that the
transaction couldn't have had the wrong witness (either because the witness was
malleated or stripped).
In preparation for wtxid-based relay, we can observe that txid == wtxid for
transactions that have no witness, and add the wtxid of rejected transactions,
provided the transaction wasn't a witness-stripped one. This means that we now
add more data to the filter (as prior to this commit, any transaction with a
witness that failed to be accepted was being skipped for inclusion in the
filter) but witness malleation should still not interfere with relay of a valid
segwit transaction, because the txid of a segwit transaction would not be added
to the filter after failing validation.
In the future, having wtxids in the recent rejects filter will allow us to
skip downloading the same wtxid multiple times, once our peers use wtxids for
transaction relay.
Diffstat (limited to 'src/protocol.cpp')
0 files changed, 0 insertions, 0 deletions