New issue
Advanced search Search tips

Issue 716207 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Apr 2017
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug-Security



Sign in to add a comment

Use-of-uninitialized-value in CFX_SeekableStreamProxy::CFX_SeekableStreamProxy

Project Member Reported by ClusterFuzz, Apr 27 2017

Issue description

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

Fuzzer: libfuzzer_pdfium_xfa_fuzzer
Job Type: libfuzzer_chrome_msan
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  CFX_SeekableStreamProxy::CFX_SeekableStreamProxy
  CFX_RetainPtr<CFX_SeekableStreamProxy> pdfium::MakeRetain<CFX_SeekableStreamProx
  CXFA_SimpleParser::StartParse
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_msan&range=467663:467695

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


Issue filed automatically.

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

Comment 1 by tsepez@chromium.org, Apr 27 2017

Owner: tsepez@chromium.org
Status: Assigned (was: Untriaged)
Seems like this can happen if the stream is too short to even contain a bom.

Comment 2 by tsepez@chromium.org, Apr 27 2017

Labels: -Security_Severity-Medium Security_Severity-Low
The way this is used, this doesn't leak any useful information.  We mask off parts of it, and then just set one of 3 values based on remainder.  

Comment 4 by tsepez@chromium.org, Apr 27 2017

Status: Fixed (was: Assigned)
Project Member

Comment 5 by ClusterFuzz, Apr 28 2017

ClusterFuzz has detected this issue as fixed in range 467695:467736.

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

Fuzzer: libfuzzer_pdfium_xfa_fuzzer
Job Type: libfuzzer_chrome_msan
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  CFX_SeekableStreamProxy::CFX_SeekableStreamProxy
  CFX_RetainPtr<CFX_SeekableStreamProxy> pdfium::MakeRetain<CFX_SeekableStreamProx
  CXFA_SimpleParser::StartParse
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_msan&range=467663:467695
Fixed: https://clusterfuzz.com/revisions?job=libfuzzer_chrome_msan&range=467695:467736

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


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 6 by sheriffbot@chromium.org, Apr 28 2017

Labels: -Restrict-View-SecurityTeam Restrict-View-SecurityNotify
Components: Internals>Plugins>PDF
Project Member

Comment 8 by sheriffbot@chromium.org, Aug 4 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
Project Member

Comment 9 by sheriffbot@chromium.org, Jul 28

Labels: Pri-2

Sign in to add a comment