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

Issue 2728103003: Add a fuzzer for png_process_data (Closed)

Created:
3 years, 9 months ago by scroggo_chromium
Modified:
3 years, 9 months ago
Reviewers:
mmoroz, Noel Gordon, inferno
CC:
chromium-reviews, fuzzing_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Add a fuzzer for png_process_data The existing png fuzzer uses png_read_rows, which is not used by any Chromium clients. Use libpng's other API for progressive reading, which is used both by PNGImageDecoder and PNGCodec. Reuses much of the code from the existing fuzzer, with a build flag to choose between them. No need to supply any callbacks to libpng. This is merely to catch errors within libpng itself. BUG=697101 Review-Url: https://codereview.chromium.org/2728103003 Cr-Commit-Position: refs/heads/master@{#454921} Committed: https://chromium.googlesource.com/chromium/src/+/11a3425fff09d249101f8fb0dcd9c00ddb80283b

Patch Set 1 #

Patch Set 2 : Rebase #

Patch Set 3 : Use same seed corpuses as the original #

Patch Set 4 : Share seed corpuses #

Patch Set 5 : More sharing #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -9 lines) Patch
M testing/libfuzzer/fuzzers/BUILD.gn View 1 2 3 4 3 chunks +20 lines, -9 lines 1 comment Download
M testing/libfuzzer/fuzzers/libpng_read_fuzzer.cc View 1 4 chunks +11 lines, -0 lines 0 comments Download

Messages

Total messages: 19 (10 generated)
scroggo_chromium
3 years, 9 months ago (2017-03-03 21:58:28 UTC) #2
inferno
lgtm
3 years, 9 months ago (2017-03-03 22:02:55 UTC) #4
Noel Gordon
lgtm
3 years, 9 months ago (2017-03-04 06:01:29 UTC) #5
mmoroz
LGTM, thanks!
3 years, 9 months ago (2017-03-05 13:47:33 UTC) #6
scroggo_chromium
PTAL - latest patch set shares the corpuses amongst the libpng fuzzers
3 years, 9 months ago (2017-03-06 18:12:06 UTC) #7
inferno
lgtm
3 years, 9 months ago (2017-03-06 18:21:28 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2728103003/80001
3 years, 9 months ago (2017-03-06 19:37:01 UTC) #15
mmoroz
LGTM https://codereview.chromium.org/2728103003/diff/80001/testing/libfuzzer/fuzzers/BUILD.gn File testing/libfuzzer/fuzzers/BUILD.gn (right): https://codereview.chromium.org/2728103003/diff/80001/testing/libfuzzer/fuzzers/BUILD.gn#newcode166 testing/libfuzzer/fuzzers/BUILD.gn:166: libpng_seed_corpuses = [ Nice! Maybe even rename it ...
3 years, 9 months ago (2017-03-06 19:37:30 UTC) #16
commit-bot: I haz the power
3 years, 9 months ago (2017-03-06 19:44:24 UTC) #19
Message was sent while issue was closed.
Committed patchset #5 (id:80001) as
https://chromium.googlesource.com/chromium/src/+/11a3425fff09d249101f8fb0dcd9...

Powered by Google App Engine
This is Rietveld 408576698