From 5e3d1e4c31e6f1134ff636ff34d5628d07d672e1 Mon Sep 17 00:00:00 2001 From: Cory Barker Date: Fri, 22 Jul 2022 17:36:14 +0000 Subject: Updated the way we build AFL++ fuzz binaries Test: Built AFL fuzzers individually and built all using haiku command and built libfuzzers individually and also by using haiku command. Ran selected fuzzers manually to ensure fuzzing still worked. Description: Previously we needed to add cc_afl_fuzz to build an afl fuzz binary, however, to turn current libFuzzers into AFL fuzzers this would required an update to each Android.bp file which is a lot of work, and would also require an approval from each Android.bp file owner, which is even more work. To get around this (and also to match how AFL fuzzers are built in G3) we will build AFL++ fuzz binaries by command line option FUZZ_FRAMEWORK. When FUZZ_FRAMEWORK=AFL is set, all cc_fuzz modules will be built Change-Id: Ic4fd34f8d5cf5cfa7db6c00235efdb230190da35 --- Android.bp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Android.bp b/Android.bp index b3bc32c..8cc9ea5 100644 --- a/Android.bp +++ b/Android.bp @@ -92,6 +92,9 @@ cc_fuzz { // exceeds the posix_spawn limit. So, only some of the files // are used. corpus: ["fuzzer/corpus/0*"], + fuzzing_frameworks: { + afl: false, + }, fuzz_config: { componentid: 87896 } -- cgit v1.2.3 From d2ebbfd4b48f2e053c5eac2c06b6b9f600218568 Mon Sep 17 00:00:00 2001 From: Cory Barker Date: Mon, 8 Aug 2022 15:55:12 +0000 Subject: Revert "Updated the way we build AFL++ fuzz binaries" Revert "Updated the way we build AFL++ fuzz binaries" Revert "Updated the way we build AFL++ fuzz binaries" Revert submission 2164482-Build AFL from Command line option Reason for revert: Unit test fails on MAC builds Reverted Changes: Ia7a822462:Updated the way we build AFL++ fuzz binaries I994bde63b:Updated the way we build AFL++ fuzz binaries Ie863853b4:Updated the way we build AFL++ fuzz binaries I577d6ae7b:Updated the way we build AFL++ fuzz binaries Ic4fd34f8d:Updated the way we build AFL++ fuzz binaries Change-Id: I2a2cfac39c2ebc8a3ba6f661b86fa4eed24517a5 --- Android.bp | 3 --- 1 file changed, 3 deletions(-) diff --git a/Android.bp b/Android.bp index 8cc9ea5..b3bc32c 100644 --- a/Android.bp +++ b/Android.bp @@ -92,9 +92,6 @@ cc_fuzz { // exceeds the posix_spawn limit. So, only some of the files // are used. corpus: ["fuzzer/corpus/0*"], - fuzzing_frameworks: { - afl: false, - }, fuzz_config: { componentid: 87896 } -- cgit v1.2.3 From 94f281830b8ffc848b988fadeab466c1e689cbe9 Mon Sep 17 00:00:00 2001 From: Cory Barker Date: Mon, 8 Aug 2022 16:47:05 +0000 Subject: Revert^2 "Updated the way we build AFL++ fuzz binaries" d2ebbfd4b48f2e053c5eac2c06b6b9f600218568 Change-Id: I0395712912d52f870820117d020f713e929c21f4 --- Android.bp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Android.bp b/Android.bp index b3bc32c..8cc9ea5 100644 --- a/Android.bp +++ b/Android.bp @@ -92,6 +92,9 @@ cc_fuzz { // exceeds the posix_spawn limit. So, only some of the files // are used. corpus: ["fuzzer/corpus/0*"], + fuzzing_frameworks: { + afl: false, + }, fuzz_config: { componentid: 87896 } -- cgit v1.2.3