New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 600709 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug



Sign in to add a comment

Crash in CPDF_Array::GetDirectObjectAt

Project Member Reported by ClusterFuzz, Apr 5 2016

Issue description

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=6521327267086336

Fuzzer: tokenfuzz_pdf_march16
Job Type: linux_asan_pdfium
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000018
Crash State:
  CPDF_Array::GetDirectObjectAt
  CPDF_DataAvail::CheckHintTables
  CPDF_DataAvail::CheckDocStatus
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_pdfium&range=357360:357514

Minimized Testcase (1171.47 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94oLhst185djeNF-QCsecmCAEPs6hLsuc-fw2Wy7VPskx48VmejQfY_Ar509p3ulI2DQMcIELocWR_Gz30aKg-j8lB-f4Cdu08_nmIRz8jwmU3BEwxl-55xL5JoT8gsnhaPBXVh6r3PPzE4DVVy0zsH9qmV5Yqt2FkmDzre5goxyzAnAE0

Filer: ivancic

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: tsepez@chromium.org ashej...@chromium.org
Owner: jun_f...@foxitsoftware.com
Status: Assigned (was: Available)


The result is a list of CLs that change the crashed files.
=========================================================

Author: Jun Fang
Component: chromium-pdfium
Changelist: https://pdfium.googlesource.com/pdfium.git/+/d946f3011984755b14d7dcfb05d572e870f93f3f
Time: Mon Nov 02 13:45:35 2015 +0800
Lines 127-128 of file fpdf_dataavail.cpp which potentially caused crash are changed in this cl (frame #5, "FPDFAvail_IsDocAvail").

File pdfium_test.cc is changed in this cl (and is part of stack frame #6, "RenderPdf")
Minimum distance from crash line to modified line: 0. (file: fpdf_dataavail.cpp, crashed on: 128, modified: 128).

Suspected Component: chromium-pdfium

@jun_fang: Hey, would you mind checking the above issue and see if it's related to your change as per the above suspected result ?

I really appreciate your help.

Thank you!
Owner: tsepez@chromium.org
Jun no longer involved with this project.
Components: Internals>Plugins>PDF
Probably the "H" key exists but it's not an array. Probably need to change the KeyExist() checks to make sure the keys are the expected types.
Possibly related:  bug 613031 ,  bug 610555 , and bug 591088.
Project Member

Comment 6 by ClusterFuzz, Jun 8 2016

ClusterFuzz has detected this issue as fixed in range 398351:398496.

Detailed report: https://cluster-fuzz.appspot.com/testcase?key=6521327267086336

Fuzzer: tokenfuzz_pdf_march16
Job Type: linux_asan_pdfium
Platform Id: linux

Crash Type: UNKNOWN READ
Crash Address: 0x000000000018
Crash State:
  CPDF_Array::GetDirectObjectAt
  CPDF_DataAvail::CheckHintTables
  CPDF_DataAvail::CheckDocStatus
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_pdfium&range=357360:357514
Fixed: https://cluster-fuzz.appspot.com/revisions?job=linux_asan_pdfium&range=398351:398496

Minimized Testcase (1171.47 Kb): https://cluster-fuzz.appspot.com/download/AMIfv95Dq2KGud7mEifQx1uemOFT0-Z5Nl26luwr4HFh9Tx9ltQchDYbPGdt_yflUYe05s1sc8dR7KnuCz_WIkxmQR_uivYXM5pQIgt5fHUEhKmNtntjxsOf--yzMZYSW-3kBENLaHKPZGSdctKxtuB9jkuPYTsC-Gpwc3X6Ls0KCHX8-dA4W20

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.

If you suspect that the result above is incorrect, try re-doing that job on the test case report page.
Project Member

Comment 7 by ClusterFuzz, Jun 14 2016

Labels: ClusterFuzz-Verified
Status: Verified (was: Assigned)
ClusterFuzz testcase is verified as fixed, closing issue.

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
Cc: mummare...@chromium.org thestig@chromium.org
 Issue 622863  has been merged into this issue.
 Issue 626718  has been merged into this issue.
Project Member

Comment 10 by sheriffbot@chromium.org, Nov 22 2016

Labels: -Restrict-View-EditIssue
Removing EditIssue view restrictions from ClusterFuzz filed bugs. If you believe that this issue should still be restricted, please reapply the label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Sign in to add a comment