New issue
Advanced search Search tips

Issue 724960 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Closed: May 2017
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

Container-overflow in CFX_UnownedPtr<unsigned char const>::Probe

Project Member Reported by ClusterFuzz, May 22 2017

Issue description

Detailed report: https://clusterfuzz.com/testcase?key=4545061466669056

Fuzzer: libfuzzer_pdfium_xfa_fuzzer
Job Type: libfuzzer_chrome_asan
Platform Id: linux

Crash Type: Container-overflow READ 1
Crash Address: 0x611000001800
Crash State:
  CFX_UnownedPtr<unsigned char const>::Probe
  CFX_SAXReader::NotifyTargetData
  CFX_SAXReader::ParseMaybeClose
  
Sanitizer: address (ASAN)

Recommended Security Severity: Medium

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_asan&range=472960:473004

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4545061466669056


Issue filed automatically.

See https://chromium.googlesource.com/chromium/src/+/master/testing/libfuzzer/reproducing.md for more information.
 
Project Member

Comment 1 by sheriffbot@chromium.org, May 22 2017

Labels: M-60
Project Member

Comment 2 by sheriffbot@chromium.org, May 22 2017

Labels: ReleaseBlock-Beta
This issue is a security regression. If you are not able to fix this quickly, please revert the change that introduced it.

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

Comment 3 by sheriffbot@chromium.org, May 22 2017

Labels: Pri-1

Comment 4 by tsepez@chromium.org, May 22 2017

Labels: -Security_Severity-Medium -Security_Impact-Head Security_Impact-None Security_Severity-Low
XFA => not shipping, Probe() => severity low.

Comment 5 by tsepez@chromium.org, May 22 2017

Owner: tsepez@chromium.org
Status: Assigned (was: Untriaged)

Comment 6 by tsepez@chromium.org, May 22 2017

CF objecting to a ptr to .data() in a vector of .size() 0.

Comment 7 by tsepez@chromium.org, May 22 2017

Status: Started (was: Assigned)
https://pdfium-review.googlesource.com/5750 is the CL.
Project Member

Comment 8 by ClusterFuzz, May 23 2017

ClusterFuzz has detected this issue as fixed in range 473689:473746.

Detailed report: https://clusterfuzz.com/testcase?key=4545061466669056

Fuzzer: libfuzzer_pdfium_xfa_fuzzer
Job Type: libfuzzer_chrome_asan
Platform Id: linux

Crash Type: Container-overflow READ 1
Crash Address: 0x611000001800
Crash State:
  CFX_UnownedPtr<unsigned char const>::Probe
  CFX_SAXReader::NotifyTargetData
  CFX_SAXReader::ParseMaybeClose
  
Sanitizer: address (ASAN)

Recommended Security Severity: Medium

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_asan&range=472960:473004
Fixed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_asan&range=473689:473746

Reproducer Testcase: https://clusterfuzz.com/download?testcase_id=4545061466669056


See https://chromium.googlesource.com/chromium/src/+/master/testing/libfuzzer/reproducing.md 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 9 by ClusterFuzz, May 23 2017

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

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
Project Member

Comment 10 by sheriffbot@chromium.org, May 23 2017

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Project Member

Comment 11 by sheriffbot@chromium.org, May 23 2017

Labels: -ReleaseBlock-Beta
Project Member

Comment 12 by sheriffbot@chromium.org, Aug 29 2017

Labels: -Restrict-View-SecurityNotify allpublic
This bug has been closed for more than 14 weeks. Removing security view restrictions.

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

Sign in to add a comment