diff options
| author | glozow <gloriajzhao@gmail.com> | 2024-04-30 09:50:03 +0100 |
|---|---|---|
| committer | glozow <gloriajzhao@gmail.com> | 2024-04-30 09:52:00 +0100 |
| commit | 36e660fc234b8c327e56144bd331b1181dc94940 (patch) | |
| tree | f67be5e16d01b41a58933b3e8411cf500d81b0fb /src/script/miniscript.cpp | |
| parent | ad42d63519839b5751db0f294b5de7cfdc332e61 (diff) | |
| parent | cc15c5bfd1eb3903de246c124702a7c66c687008 (diff) | |
Merge bitcoin/bitcoin#29990: fuzz: don't allow adding duplicate transactions to the mempool
cc15c5bfd1eb3903de246c124702a7c66c687008 fuzz: don't allow adding duplicate transactions to the mempool (Suhas Daftuar)
Pull request description:
Filter duplicate transaction ids from being added to the mempool in the `partially_downloaded_block` fuzz target.
I think a prerequisite for calling `CTxMemPool::addUnchecked` should be that the underlying txid doesn't already exist in the mempool (otherwise `addUnchecked` would need a way to return failure, which we don't currently have).
ACKs for top commit:
glozow:
utACK cc15c5bfd1eb3903de246c124702a7c66c687008 makes sense to me
maflcko:
lgtm ACK cc15c5bfd1eb3903de246c124702a7c66c687008
brunoerg:
ACK cc15c5bfd1eb3903de246c124702a7c66c687008
dergoegge:
utACK cc15c5bfd1eb3903de246c124702a7c66c687008
Tree-SHA512: 85f84ce405aba584e6d00391515f0a86c5648ce8b2da69036e50a6c1f6833d050d09b1972cc5ffbe7c4edb3e5f7f965ef34bd839deeddac27a889cc8d2e53b8f
Diffstat (limited to 'src/script/miniscript.cpp')
0 files changed, 0 insertions, 0 deletions