Star1
23
代码
代码
Issues16
Pull Requests19
流水线
项目讨论
Wiki
分析
Star1
23
ohci1ohci1Revert "!44 merge 144_trunk into 144_trunk"
148e6fa5创建于 1月10日历史提交
文件最后提交记录最后更新时间
BUILD.gn
[exit-time-destructors] Exclude warning from targets This CL is part of number of CLs disabling -Wexit-time-destructorsin places where this warning is going off. This will allow having this warning enabled-by-default across the codebase, whilst documenting the places where corrections for this warning are necessary. Bug: 40031409 Change-Id: I1003733d31268bdf7bf4f90288de9c8f61b9e468 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/134610 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
11 个月前
DEPS
Avoid setting DiscardableMemoryAllocator in LLVMFuzzerTestOneInput() If compiled with chromium, a DiscardableMemoryAllocator is already set by LLVMFuzzerInitialize() in chromium before calling LLVMFuzzerTestOneInput() when running pdfium_fuzzer, which causes DCHECK failure in DiscardableMemoryAllocator::SetInstance() since the allocator is already set. This CL removes the steps of setting up DiscardableMemoryAllocator in LLVMFuzzerTestOneInput() to avoid the DCHECK failure. And it simplifies the fuzzer process to only set up the DiscardableMemoryAllocator once. Bug: chromium:1440491 Change-Id: I48e678253dacd696bd0fbf01805e150d9026ac53 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/107890 Commit-Queue: Nigi <nigi@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
3 年前
component_fuzzer_template.cc
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_bidi_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_cfgas_stringformatter_fuzzer.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdf_cfx_barcode_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_cjs_util_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_cmap_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_codec_a85_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_codec_bmp_fuzzer.cc
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_codec_fax_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_codec_gif_fuzzer.cc
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_codec_icc_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_codec_jbig2_fuzzer.cc
Undo incorrect arithmetic decoder change + follow-up Reverts https://pdfium-review.googlesource.com/c/pdfium/+/52712 Reverts https://pdfium-review.googlesource.com/c/pdfium/+/16791 The last of those two is incorrect per JBIG2 spec, the first papers over an issue caused by it. The last change even cites the spec part that says that an arithmetic decoder should keep returning 1-bits on out-of-data. This is also described in detail in Annex E. This includes two tests, one of which has several trailing 7fff byte pairs stripped before the final ffac pair. pdf.js, Preview.app, Acrobat Reader all decode these files fine. The construction of these tests is described in detail in: * https://github.com/SerenityOS/serenity/pull/26272/commits/ba49ffac4dd * https://github.com/SerenityOS/serenity/pull/26242/commits/9882a8462b6 I took these two files, converted them to PDF using Meta/jbig2_to_pdf.py in that repo, and then to .in format using the script in https://pdfium-review.googlesource.com/c/pdfium/+/136590 This also fixes decoding 042_25.jb2 from the Power JBIG2 test set. clusterfuzz-testcase-pdf_codec_jbig2_fuzzer-5210024078213120 from the third linked bug is a 5636 x 60419 image and needs ~35s to run on my machine. To not re-introduce the timeouts fixed by the incorrect fix, instead limit regions to 4096 x 4096 while fuzzing. This makes pdf_codec_jbig2_fuzzer process that test case in 0 ms instead of in 35s on my machine as it now early-outs. I confirmed that bug_947622_min.pdf from comment 7 on bug 41450095 renders exactly the same before and after this change. Bug: 432920906,41450095,40540933 Change-Id: Idc5c13c9440d0b4749f0832659bcdf36c1bc3597 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/136630 Reviewed-by: Nico Weber <thakis@google.com> Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
9 个月前
pdf_codec_jpeg_fuzzer.cc
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_codec_png_fuzzer.cc
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_codec_rle_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_codec_tiff_fuzzer.cc
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_cpdf_tounicodemap_fuzzer.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdf_css_fuzzer.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdf_font_fuzzer.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdf_formcalc_context_fuzzer.cc
Convert to google_style_ members in xfa/ -- Includes sub-directories. -- Manual changes required at boundaries between old and new code. Bug: 42271580 Change-Id: I1bc9e871414b4b7bfbf0335a4d5cbd6275e0591d Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130330 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: Tom Sepez <tsepez@chromium.org>
1 年前
pdf_formcalc_fuzzer.cc
Remove static Create() helper methods from String Views Follow-on from https://pdfium-review.googlesource.com/c/pdfium/+/121150 but performs the same change on non-owning string view types. Change-Id: I3f5de339401750ed767073c095edcbfdc09f1c77 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/121119 Reviewed-by: Lei Zhang <thestig@chromium.org> Reviewed-by: Tom Sepez <tsepez@google.com> Commit-Queue: Tom Sepez <tsepez@chromium.org>
2 年前
pdf_formcalc_translate_fuzzer.cc
Remove static Create() helper methods from String Views Follow-on from https://pdfium-review.googlesource.com/c/pdfium/+/121150 but performs the same change on non-owning string view types. Change-Id: I3f5de339401750ed767073c095edcbfdc09f1c77 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/121119 Reviewed-by: Lei Zhang <thestig@chromium.org> Reviewed-by: Tom Sepez <tsepez@google.com> Commit-Queue: Tom Sepez <tsepez@chromium.org>
2 年前
pdf_fuzzer_init.cc
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_fuzzer_init_public.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdf_fuzzer_init_public.h
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_fuzzer_templates.h
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前
pdf_fx_date_helpers_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_hint_table_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_jpx_fuzzer.cc
Make pdf_jpx_fuzzer more realistic Currently, pdf_jpx_fuzzer tests CJPX_Decoder::Decode() with some bespoke code to determine the bitmap format and the corresponding image component count. Replace this code with JpxDecodeConversion, to better align the fuzzer with the logic in the production CJPX_Decoder::Decode() caller. As a result: 1) This potentially can avoid some fuzzing failures that may not be possible with production code. 2) This adds support for fuzzing CJPX_Decoder::Decode() with varying swap_rgb values. Currently, this argument is hard-coded to false. Bug: 428771937 Change-Id: Iadbc858024e5cc46d20ed429b0716cb734845c8b Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/135550 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
10 个月前
pdf_lzw_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_nametree_fuzzer.cc
Revert "!44 merge 144_trunk into 144_trunk" This reverts commit 20970951ddb733e75f4db5c7a7e0829a18c874db, reversing changes made to 66c6bc40966122935d37eef739deb988581214d4. Signed-off-by: ohci1 <ohci1@noreply.gitcode.com>
6 个月前
pdf_psengine_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_scanlinecompositor_fuzzer.cc
Revert "!44 merge 144_trunk into 144_trunk" This reverts commit 20970951ddb733e75f4db5c7a7e0829a18c874db, reversing changes made to 66c6bc40966122935d37eef739deb988581214d4. Signed-off-by: ohci1 <ohci1@noreply.gitcode.com>
6 个月前
pdf_streamparser_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdf_xfa_fdp_fuzzer.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdf_xfa_raw_fuzzer.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdf_xfa_xdp_fdp_fuzzer.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdf_xml_fuzzer.cc
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
pdfium_fuzzer.cc
Avoid setting DiscardableMemoryAllocator in LLVMFuzzerTestOneInput() If compiled with chromium, a DiscardableMemoryAllocator is already set by LLVMFuzzerInitialize() in chromium before calling LLVMFuzzerTestOneInput() when running pdfium_fuzzer, which causes DCHECK failure in DiscardableMemoryAllocator::SetInstance() since the allocator is already set. This CL removes the steps of setting up DiscardableMemoryAllocator in LLVMFuzzerTestOneInput() to avoid the DCHECK failure. And it simplifies the fuzzer process to only set up the DiscardableMemoryAllocator once. Bug: chromium:1440491 Change-Id: I48e678253dacd696bd0fbf01805e150d9026ac53 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/107890 Commit-Queue: Nigi <nigi@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
3 年前
pdfium_fuzzer_helper.cc
Limit bitmap memory usage in PDFiumFuzzerHelper The fuzzer may try to create very large bitmaps and reach an OOM condition. Limit the amount of memory used for bitmap allocation to avoid this. Bug: 458896741 Change-Id: I7820d27faa373e2ee1633c9af7e0cc776e64e678 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/137891 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
8 个月前
pdfium_fuzzer_helper.h
Change pdfium_fuzzer_helper.cc to render with different bitmap formats Extract more hashes from the input and use that as the bitmap format. Switch from FPDFBitmap_Create() to FPDFBitmap_CreateEx() to put the bitmap format variable to use. Change-Id: I2edefb0f9df60c2bec7b8a02f0189b667c6ce4d5 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/137692 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
8 个月前
pdfium_fuzzer_util.cc
Fix non-compliant copyright headers Fixes copyright headers that neither match the old nor the new style. In most of these cases, this is because the copyright header was copied from another project, such as Chromium. Bug: pdfium:1884 Change-Id: I1943df9c2f8a0282be9f7691bdd38b054d237458 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100391 Auto-Submit: K. Moon <kmoon@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
3 年前
pdfium_fuzzer_util.h
Fix non-compliant copyright headers Fixes copyright headers that neither match the old nor the new style. In most of these cases, this is because the copyright header was copied from another project, such as Chromium. Bug: pdfium:1884 Change-Id: I1943df9c2f8a0282be9f7691bdd38b054d237458 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100391 Auto-Submit: K. Moon <kmoon@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
3 年前
pdfium_xfa_fuzzer.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
pdfium_xfa_lpm_fuzz_stub.cc
Fix non-compliant copyright headers Fixes copyright headers that neither match the old nor the new style. In most of these cases, this is because the copyright header was copied from another project, such as Chromium. Bug: pdfium:1884 Change-Id: I1943df9c2f8a0282be9f7691bdd38b054d237458 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100391 Auto-Submit: K. Moon <kmoon@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
3 年前
pdfium_xfa_lpm_fuzz_stub.h
Fix non-compliant copyright headers Fixes copyright headers that neither match the old nor the new style. In most of these cases, this is because the copyright header was copied from another project, such as Chromium. Bug: pdfium:1884 Change-Id: I1943df9c2f8a0282be9f7691bdd38b054d237458 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100391 Auto-Submit: K. Moon <kmoon@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
3 年前
xfa_codec_fuzzer.h
Replace calls to pdfium::make_span() with CTAD. Deduction rules are imperfect, and we need to provide more information than ought be required in some cases. In particular, we need to call out dynamic_spans in a few places where spans of different sizes are combined by a questionmark operator. Bug: 42271100 Change-Id: Ic5299c1331702e84bcbb2b3cef1e731d00c06f03 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/131172 Commit-Queue: Tom Sepez <tsepez@chromium.org> Reviewed-by: Lei Zhang <thestig@chromium.org>
1 年前
xfa_process_state.cc
Clang-format //testing Change-Id: I27360f1264aefc45286e07242f30f1882aeca2ba Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/130753 Reviewed-by: Tom Sepez <tsepez@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
1 年前
xfa_process_state.h
Update old-style copyright headers Updates old-style copyright headers to the new style, by removing "All rights reserved." Also inserts "The" before "PDFium Authors", which is required by both styles. Mechanically generated by this command: PATTERN='Copyright \([0-9]\+\) \(The \)\?PDFium Authors. All rights reserved.' git grep -l "$PATTERN" \ | xargs sed "s/$PATTERN/Copyright \1 The PDFium Authors/" -i'' Fixed: pdfium:1884 Change-Id: I6f781d811df8839e2e21b85f716529b813828bcd Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/100371 Reviewed-by: Lei Zhang <thestig@chromium.org> Commit-Queue: K. Moon <kmoon@chromium.org>
3 年前