Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(87)

Issue 2075883002: Add American Fuzzy Lop (afl) to third_party/afl/ (Closed)

Created:
4 years, 6 months ago by Jonathan Metzman
Modified:
4 years, 6 months ago
Reviewers:
jam, aarya, inferno
CC:
chromium-reviews, kcc2, aizatsky, mmoroz
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add American Fuzzy Lop (afl) to third_party/afl/ Exclude the following unneeded files/directories from afl: 1. src/docs/visualization/ 2. src/docs/vuln_samples/ 3. src/testcases/ 4. third_party/afl/src/experimental/argv_fuzzing/argv-fuzz-inl.h 5. third_party/afl/src/llvm_mode/afl-llvm-pass.so.cc 6. third_party/afl/src/hash.h 7. third_party/afl/experimental/instrumented_cmp/instrumented_cmp.c Directories 1-3 contain binary files. We will exclude these files once we start using DEPS by reorganizing afl upstream so that source files are kept in a src/ directory. The other directories won't be cloned by DEPS. Files 4-5 contain includes that cause checkdeps to complain when git cl presubmit is run. Once we start using DEPS this will be handled through exclude rules in DEPS. Files 6-7 do not have license headers and fail checklicenses when git cl presubmit is run. This will be fixed upstream by adding license headers. DEPS will be used for afl once it gets a repo. BUG=611337 Committed: https://crrev.com/2c059446f38a735223d380a151c281a70e1643ca Cr-Commit-Position: refs/heads/master@{#401096}

Patch Set 1 #

Patch Set 2 : Add AFL to third_party/afl/. #

Patch Set 3 : Add American Fuzzy Lop (afl) to third_party/afl/ #

Total comments: 3

Patch Set 4 : Fix nits #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22523 lines, --2 lines) Patch
A third_party/afl/BUILD.gn View 1 2 3 1 chunk +27 lines, -0 lines 0 comments Download
A + third_party/afl/OWNERS View 0 chunks +-1 lines, --1 lines 0 comments Download
A third_party/afl/README.chromium View 1 2 1 chunk +23 lines, -0 lines 0 comments Download
A third_party/afl/src/Makefile View 1 chunk +146 lines, -0 lines 0 comments Download
A third_party/afl/src/QuickStartGuide.txt View 1 chunk +1 line, -0 lines 0 comments Download
A third_party/afl/src/README View 1 chunk +1 line, -0 lines 0 comments Download
A third_party/afl/src/afl-analyze.c View 1 chunk +1065 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-as.h View 1 chunk +717 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-as.c View 1 chunk +544 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-cmin View 1 chunk +447 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-fuzz.c View 1 chunk +7861 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-gcc.c View 1 chunk +318 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-gotcpu.c View 1 chunk +252 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-plot View 1 chunk +166 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-showmap.c View 1 chunk +728 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-tmin.c View 1 chunk +1086 lines, -0 lines 0 comments Download
A third_party/afl/src/afl-whatsup View 1 chunk +163 lines, -0 lines 0 comments Download
A third_party/afl/src/alloc-inl.h View 1 chunk +570 lines, -0 lines 0 comments Download
A third_party/afl/src/config.h View 1 chunk +344 lines, -0 lines 0 comments Download
A third_party/afl/src/debug.h View 1 chunk +251 lines, -0 lines 0 comments Download
A + third_party/afl/src/docs/COPYING View 0 chunks +-1 lines, --1 lines 0 comments Download
A third_party/afl/src/docs/ChangeLog View 1 chunk +2279 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/INSTALL View 1 chunk +169 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/QuickStartGuide.txt View 1 chunk +48 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/README View 1 chunk +466 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/env_variables.txt View 1 chunk +219 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/historical_notes.txt View 1 chunk +147 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/notes_for_asan.txt View 1 chunk +125 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/parallel_fuzzing.txt View 1 chunk +202 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/perf_tips.txt View 1 chunk +212 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/sister_projects.txt View 1 chunk +275 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/status_screen.txt View 1 chunk +389 lines, -0 lines 0 comments Download
A third_party/afl/src/docs/technical_details.txt View 1 chunk +515 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/README.experiments View 1 chunk +42 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/asan_cgroups/limit_memory.sh View 1 chunk +157 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/bash_shellshock/shellshock-fuzz.diff View 1 chunk +59 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/canvas_harness/canvas_harness.html View 1 chunk +170 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/clang_asm_normalize/as View 1 chunk +75 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/crash_triage/triage_crashes.sh View 1 chunk +89 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/distributed_fuzzing/sync_script.sh View 1 chunk +89 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/libpng_no_checksum/libpng-nocrc.patch View 1 chunk +15 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/persistent_demo/persistent_demo.c View 1 chunk +89 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/post_library/post_library.so.c View 1 chunk +119 lines, -0 lines 0 comments Download
A third_party/afl/src/experimental/post_library/post_library_png.so.c View 1 chunk +113 lines, -0 lines 0 comments Download
A third_party/afl/src/llvm_mode/Makefile View 1 chunk +112 lines, -0 lines 0 comments Download
A third_party/afl/src/llvm_mode/README.llvm View 1 chunk +196 lines, -0 lines 0 comments Download
A third_party/afl/src/llvm_mode/afl-clang-fast.c View 1 chunk +327 lines, -0 lines 0 comments Download
A third_party/afl/src/llvm_mode/afl-llvm-rt.o.c View 1 chunk +283 lines, -0 lines 0 comments Download
A third_party/afl/src/qemu_mode/README.qemu View 1 chunk +120 lines, -0 lines 0 comments Download
A third_party/afl/src/qemu_mode/build_qemu_support.sh View 1 chunk +196 lines, -0 lines 0 comments Download
A third_party/afl/src/qemu_mode/patches/afl-qemu-cpu-inl.h View 1 chunk +294 lines, -0 lines 0 comments Download
A third_party/afl/src/qemu_mode/patches/cpu-exec.diff View 1 chunk +33 lines, -0 lines 0 comments Download
A third_party/afl/src/qemu_mode/patches/elfload.diff View 1 chunk +32 lines, -0 lines 0 comments Download
A third_party/afl/src/qemu_mode/patches/syscall.diff View 1 chunk +25 lines, -0 lines 0 comments Download
A third_party/afl/src/qemu_mode/patches/translate-all.diff View 1 chunk +18 lines, -0 lines 0 comments Download
A third_party/afl/src/test-instr.c View 1 chunk +37 lines, -0 lines 0 comments Download
A third_party/afl/src/types.h View 1 chunk +79 lines, -0 lines 0 comments Download

Messages

Total messages: 20 (11 generated)
inferno
lgtm with nits. https://codereview.chromium.org/2075883002/diff/40001/third_party/afl/BUILD.gn File third_party/afl/BUILD.gn (right): https://codereview.chromium.org/2075883002/diff/40001/third_party/afl/BUILD.gn#newcode5 third_party/afl/BUILD.gn:5: source_set("afl_runtime") { s/afl_runtime/afl https://codereview.chromium.org/2075883002/diff/40001/third_party/afl/BUILD.gn#newcode9 third_party/afl/BUILD.gn:9: cflags ...
4 years, 6 months ago (2016-06-21 14:53:18 UTC) #3
inferno
4 years, 6 months ago (2016-06-21 15:05:51 UTC) #5
inferno
lgtm John - can you please do the owners review. This is used for fuzzing, ...
4 years, 6 months ago (2016-06-21 15:19:19 UTC) #6
jam
On 2016/06/21 15:19:19, inferno wrote: > lgtm > > John - can you please do ...
4 years, 6 months ago (2016-06-21 19:09:14 UTC) #7
aarya
On 2016/06/21 19:09:14, jam wrote: > On 2016/06/21 15:19:19, inferno wrote: > > lgtm > ...
4 years, 6 months ago (2016-06-21 19:16:38 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2075883002/60001
4 years, 6 months ago (2016-06-21 19:17:23 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2075883002/60001
4 years, 6 months ago (2016-06-21 19:18:15 UTC) #15
commit-bot: I haz the power
Committed patchset #4 (id:60001)
4 years, 6 months ago (2016-06-21 20:35:00 UTC) #18
commit-bot: I haz the power
4 years, 6 months ago (2016-06-21 20:37:36 UTC) #20
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/2c059446f38a735223d380a151c281a70e1643ca
Cr-Commit-Position: refs/heads/master@{#401096}

Powered by Google App Engine
This is Rietveld 408576698