No description provided.
Delete non-reduced fuzz inputs #44
pull maflcko wants to merge 3 commits into bitcoin-core:master from bitcoin-core:2101-deleteInputs changing 0 files +0 −0-
maflcko commented at 5:18 PM on January 27, 2021: contributor
-
Delete fuzz inputs cf73e93df4
-
Reduced inputs for fuzzer 80598c15eb
-
Reduced inputs for fuzzer,address,undefined,integer ae401aa90e
-
maflcko commented at 5:20 PM on January 27, 2021: contributor
Can be "reproduced" with:
apt update && apt install wget -y wget https://raw.githubusercontent.com/bitcoin-core/bitcoin-maintainer-tools/master/delete_nonreduced_fuzz_inputs.sh bash delete_nonreduced_fuzz_inputs.sh -
laanwj commented at 8:16 PM on January 27, 2021: member
Github is completely baffled by the size of the diff :see_no_evil:
-
maflcko commented at 6:53 AM on January 28, 2021: contributor
I ran this twice and the difference was 7000 files, which is expected, because our fuzz targets are not deterministic
-
fanquake commented at 10:03 AM on January 28, 2021: member
Would be nice if someone could test #44
I ran this and it seemed to achieve something similar to this PR; reducing the seed count by ~80'000:
# master @ 3038a21dd7d6dfb3574401b8c76e48ca87c44769 find . -type f | wc -l 241321 ./ delete_nonreduced_fuzz_inputs.sh # git log commit f08f63d6121d794b18213f71ead508649055a493 (HEAD -> master) Author: delete_nonreduced_inputs script <noreply@noreply.noreply> Date: Thu Jan 28 08:47:11 2021 +0000 Reduced inputs for fuzzer,address,undefined,integer commit 12116b05479ba817f0b9c98e00ae3ddf47f0fbe3 Author: delete_nonreduced_inputs script <noreply@noreply.noreply> Date: Thu Jan 28 07:42:43 2021 +0000 Reduced inputs for fuzzer commit c9e492d4d9c1281aeb107b027205f895b35d6ec9 Author: delete_nonreduced_inputs script <noreply@noreply.noreply> Date: Thu Jan 28 07:26:52 2021 +0000 Delete fuzz inputs commit 3038a21dd7d6dfb3574401b8c76e48ca87c44769 (origin/master, origin/HEAD) # find . -type f | wc -l (my reduced branch) 163311 # find . -type f | wc -l ( this branch @ ae401aa90e5e3fa996b4066cb067f23a26e51a4d) 163242 -
maflcko commented at 11:06 AM on January 28, 2021: contributor
Ok, going to merge. If something went wrong, it is easy to revert.
- maflcko merged this on Jan 28, 2021
- maflcko closed this on Jan 28, 2021
- maflcko deleted the branch on Jan 28, 2021
-
practicalswift commented at 9:10 PM on January 28, 2021: contributor
Post-merge ACK ae401aa90e5e3fa996b4066cb067f23a26e51a4d: nice! :)
- maflcko referenced this in commit dd0521b640 on Feb 11, 2021
- sidhujag referenced this in commit 189d0152ee on Feb 11, 2021
- PastaPastaPasta referenced this in commit 872def690b on Feb 6, 2023
- PastaPastaPasta referenced this in commit e9e79ebf16 on Apr 6, 2023
- PastaPastaPasta referenced this in commit 63d0471244 on Apr 6, 2023
- PastaPastaPasta referenced this in commit 3a59c53cad on Apr 9, 2023