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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2070103002: Add CFX_SAXReader fuzzer (Closed)

Created:
4 years, 6 months ago by dsinclair
Modified:
4 years, 6 months ago
Reviewers:
Lei Zhang, Oliver Chang
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Project:
pdfium
Visibility:
Public.

Description

Add CFX_SAXReader fuzzer This CL adds a fuzzer for the CFX_SAXReader. BUG=chromium:587126 Committed: https://pdfium.googlesource.com/pdfium/+/5377267504015d056bc0860ffadc23289b21039d

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -10 lines) Patch
M testing/libfuzzer/BUILD.gn View 1 chunk +14 lines, -0 lines 0 comments Download
M testing/libfuzzer/fuzzers.gyp View 1 chunk +11 lines, -0 lines 0 comments Download
A + testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc View 2 chunks +16 lines, -10 lines 2 comments Download

Messages

Total messages: 9 (4 generated)
dsinclair
PTAL. https://codereview.chromium.org/2070103002/diff/20001/testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc File testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc (right): https://codereview.chromium.org/2070103002/diff/20001/testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc#newcode30 testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc:30: while (1) { Will clusterfuzz time this out ...
4 years, 6 months ago (2016-06-15 17:54:32 UTC) #3
Lei Zhang
lgtm
4 years, 6 months ago (2016-06-15 19:29:56 UTC) #4
Oliver Chang
lgtm https://codereview.chromium.org/2070103002/diff/20001/testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc File testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc (right): https://codereview.chromium.org/2070103002/diff/20001/testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc#newcode30 testing/libfuzzer/pdf_cfx_saxreader_fuzzer.cc:30: while (1) { On 2016/06/15 17:54:32, dsinclair wrote: ...
4 years, 6 months ago (2016-06-15 22:43:44 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2070103002/20001
4 years, 6 months ago (2016-06-16 14:23:20 UTC) #7
commit-bot: I haz the power
4 years, 6 months ago (2016-06-16 14:40:51 UTC) #9
Message was sent while issue was closed. 
Committed patchset #1 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/5377267504015d056bc0860ffadc23289b21...

Powered by Google App Engine
This is Rietveld 408576698