Crash in base::win::ForceCrashOnSigAbort |
|||||||||||
Issue descriptionDetailed report: https://cluster-fuzz.appspot.com/testcase?key=4921097789374464 Fuzzer: ochang_neurofuzz_borgfuzz Job Type: windows_asan_chrome Platform Id: windows Crash Type: UNKNOWN WRITE Crash Address: 0x00000000 Crash State: base::win::ForceCrashOnSigAbort CPDF_HintTables::ReadPageHintTable CPDF_HintTables::LoadHintStream Regressed: https://cluster-fuzz.appspot.com/revisions?job=windows_asan_chrome&range=414945:414952 Unminimized Testcase: https://cluster-fuzz.appspot.com/download/AMIfv97vgLOh4o68HPLohEbHRrx5syCz9EJIgpU9dnYcuGV3j16EwJkaoDxfxHt37-JlV020bz8b-TJVDEswfyadlM4lOZWaFyJN9cHVtW2rU_xbrbItGOgWy1mdnA_RzFHXHeOM9hQatLrEXrhm1VPvfDDzepsMd856fP00cC56QmL8cjA54ts?testcase_id=4921097789374464 Issue manually filed by: msrchandra See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
,
Aug 29 2016
,
Aug 30 2016
dsinclair@ could you please look into this.please feel free to re-assigned back if needed. thanks in advance !
,
Aug 30 2016
,
Aug 31 2016
Passing to thestig@ who has been looking at CPDF_HintsTable crashes.
,
Aug 31 2016
,
Sep 1 2016
This crash doesn't seem to make sense. We have: 229: hStream->SkipBits(safeTotalPageLen.ValueOrDie()); 230: hStream->ByteAlign(); How can we nullptr deref on line 230? I can't repro the crash on Linux Chrome, or with Linux ASAN + pdfium_test.
,
Sep 1 2016
Though it does crash on Windows Chrome. Interesting.
,
Sep 1 2016
Maybe the symbolization is screwed up because of inlining, and we eventually abort on a failed safeTotalPageLen.ValueOrDie() on line 229?
,
Sep 1 2016
oh, nevermind, that shouldn't happen because of the CanReadFromBitStream check before
,
Sep 1 2016
https://codereview.chromium.org/2300903002/ - only happens on Windows because FX_FILESIZE is only an int32_t. We may want to consider changing that someday if we ever want to support very large PDFs. The Books team have a few samples.
,
Sep 1 2016
,
Sep 1 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/dccbd50cab878f34db9f5d4df2642ca1a2ab368e commit dccbd50cab878f34db9f5d4df2642ca1a2ab368e Author: pdfium-deps-roller <pdfium-deps-roller@chromium.org> Date: Thu Sep 01 20:19:24 2016 Roll src/third_party/pdfium/ 5e2d5c7ca..8d3ca1484 (1 commit). https://pdfium.googlesource.com/pdfium.git/+log/5e2d5c7ca2d0..8d3ca14840a0 $ git log 5e2d5c7ca..8d3ca1484 --date=short --no-merges --format='%ad %ae %s' 2016-09-01 thestig Handle another integer overflow in ReadPageHintTable(). BUG= 641882 TBR=dsinclair@chromium.org Review-Url: https://codereview.chromium.org/2301173002 Cr-Commit-Position: refs/heads/master@{#416031} [modify] https://crrev.com/dccbd50cab878f34db9f5d4df2642ca1a2ab368e/DEPS
,
Oct 18 2016
,
Nov 22 2016
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 |
|||||||||||
Comment 1 by msrchandra@chromium.org
, Aug 29 2016Components: Tools>Test>FindIt>NoResult
Labels: Needs-triage Te-Logged