Add fuzzer for jbig2 parsing

Add fuzzer for jbig2 parsing

Committed: https://pdfium.googlesource.com/pdfium/+/ec7a9455c15b2cebb75a6036c8636beb601e543a

[kcwu, 2016-09-27 16:33:35]

kcwu@chromium.org changed reviewers:
+ thestig@chromium.org

[kcwu, 2016-09-27 16:33:35]

p.s. I choice you as reviewer because you are the only common owner of pdf/ and
third_party/pdfium.

Please let me know if you'd like to share review loading with other owners.

[Lei Zhang, 2016-09-27 16:39:54]

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
File testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc (right):

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc:21: return 0;
nit: blank line after, just like line 29-30.

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc:31: std::unique_ptr<CPDF_Object>
stream(new CPDF_Stream);
CPDF_Object probably needs a ReleaseDeleter.

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc:33: CCodec_Jbig2Context
jbig2context;
nit "jbig2_context" ?

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc:33: CCodec_Jbig2Context
jbig2context;
Declare this var and |document_context| below closer to where its first used.

[kcwu, 2016-09-27 17:03:08]

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
File testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc (right):

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc:21: return 0;
On 2016/09/27 16:39:54, Lei Zhang wrote:
> nit: blank line after, just like line 29-30.

Done.

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc:31: std::unique_ptr<CPDF_Object>
stream(new CPDF_Stream);
Ah, you are right.
I saw std::default_delete<CPDF_Object> mentioned here
https://cs.chromium.org/chromium/src/third_party/pdfium/core/fpdfapi/fpdf_par...
but it is not actually declared.

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc:33: CCodec_Jbig2Context
jbig2context;
On 2016/09/27 16:39:54, Lei Zhang wrote:
> nit "jbig2_context" ?

Done.

https://codereview.chromium.org/2370943004/diff/1/testing/libfuzzer/pdf_codec...
testing/libfuzzer/pdf_codec_jbig2_fuzzer.cc:33: CCodec_Jbig2Context
jbig2context;
On 2016/09/27 16:39:54, Lei Zhang wrote:
> Declare this var and |document_context| below closer to where its first used.

Done.

[Lei Zhang, 2016-09-27 17:05:48]

lgtm

[kcwu, 2016-09-27 17:06:30]

The CQ bit was checked by kcwu@chromium.org

[commit-bot: I haz the power, 2016-09-27 17:06:33]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-09-27 21:06:51]

Description was changed from

==========
Add fuzzer for jbig2 parsing
==========

to

==========
Add fuzzer for jbig2 parsing

Committed:
https://pdfium.googlesource.com/pdfium/+/ec7a9455c15b2cebb75a6036c8636beb601e...
==========

[commit-bot: I haz the power, 2016-09-27 21:06:53]

Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/ec7a9455c15b2cebb75a6036c8636beb601e...