fuzz: Remove addrdb fuzz target #22853

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:2109-fuzzRemT changing 2 files +0 −24

MarcoFalke commented at 7:33 AM on September 1, 2021: member
The target has several issues:
- It is named incorrectly (addrdb, but it constructs a CBanEntry)
- It doesn't do anything meaningful, other than consuming one integer and passing it to a constructor
- It consumes CPU time that can be used for the other targets
- It is redundant with the banman fuzz target
Fix all by removing it.
fanquake added the label Tests on Sep 1, 2021
fuzz: Remove addrdb fuzz target fa18553d38
MarcoFalke force-pushed on Sep 1, 2021
amitiuttarwar commented at 3:51 PM on September 1, 2021: contributor

ACK fa18553d382a7d8c447cd6698b36e293fb7ecf1f, thanks for the cleanup
fanquake merged this on Sep 2, 2021
fanquake closed this on Sep 2, 2021
sidhujag referenced this in commit 1242ed1e09 on Sep 2, 2021
MarcoFalke deleted the branch on Sep 2, 2021
DrahtBot locked this on Sep 2, 2022

Contributors

Labels