New issue
Advanced search Search tips

Issue 668851 link

Starred by 1 user

Issue metadata

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

Blocking:
issue 62400



Sign in to add a comment

Use-of-uninitialized-value in tiff_read

Project Member Reported by ClusterFuzz, Nov 27 2016

Issue description

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

Fuzzer: libfuzzer_pdf_codec_tiff_fuzzer
Job Type: libfuzzer_chrome_msan
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  tiff_read
  OJPEGReadHeaderInfoSecTablesAcTable
  OJPEGReadHeaderInfoSec
  
Recommended Security Severity: Medium

Regressed: https://cluster-fuzz.appspot.com/revisions?job=libfuzzer_chrome_msan&range=427783:427885

Minimized Testcase (0.50 Kb): https://cluster-fuzz.appspot.com/download/AMIfv96jdmog8BbbFQdK1okVH9YpFMtXs6iU8AV4BvgpR7Jl1Pj7Ka5HR8gEKJaPAewCsn7LHZXngM_Ux2sO27_GEgKpZsY8DMMsaYI4dRSIrk3eMNViT7tHNCkBjQn2ywSMnt4rCPCR22O_hmYesFuBVyi0MDWpKw?testcase_id=4645927739719680

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, Nov 27 2016

Labels: M-56
Project Member

Comment 2 by sheriffbot@chromium.org, Nov 27 2016

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, Nov 27 2016

Labels: Pri-1
Components: Internals>Plugins>PDF
Owner: tsepez@chromium.org
Status: Assigned (was: Untriaged)
+tsepez: looks like thestig might have introduced this one in https://codereview.chromium.org/2456553002. Do you mind taking a look?

Comment 5 by tsepez@chromium.org, Nov 28 2016

Status: Started (was: Assigned)

Comment 6 by tsepez@chromium.org, Nov 28 2016

Cc: tsepez@chromium.org
Owner: jsc...@chromium.org
Justin, this is the PDF-copy of safe math, does this ring any bells for something you might have fixed in //base ?

Comment 7 by jsc...@chromium.org, Nov 28 2016

Sorry, I can't imagine how this could be getting an uninitialized value, since the initialization and then an assignment are immediately preceding the failing line. Could there be an msan bug here?
Project Member

Comment 8 by sheriffbot@chromium.org, Dec 2 2016

Labels: -Security_Impact-Head Security_Impact-Beta
Any updates on this?  We're planning on shipping to Beta tomorrow on 12/8
Blocking: 62400
Labels: -ReleaseBlock-Beta -M-56 -Security_Impact-Beta Security_Impact-None
This is part of XFA and not enabled on any branch of Chrome.
Project Member

Comment 11 by ClusterFuzz, Jan 12 2017

ClusterFuzz has detected this issue as fixed in range 443057:443122.

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

Fuzzer: libfuzzer_pdf_codec_tiff_fuzzer
Job Type: libfuzzer_chrome_msan
Platform Id: linux

Crash Type: Use-of-uninitialized-value
Crash Address: 
Crash State:
  tiff_read
  OJPEGReadHeaderInfoSecTablesAcTable
  OJPEGReadHeaderInfoSec
  
Sanitizer: memory (MSAN)

Recommended Security Severity: Medium

Regressed: https://cluster-fuzz.appspot.com/revisions?job=libfuzzer_chrome_msan&range=427783:427885
Fixed: https://cluster-fuzz.appspot.com/revisions?job=libfuzzer_chrome_msan&range=443057:443122

Minimized Testcase (0.50 Kb): https://cluster-fuzz.appspot.com/download/AMIfv96jdmog8BbbFQdK1okVH9YpFMtXs6iU8AV4BvgpR7Jl1Pj7Ka5HR8gEKJaPAewCsn7LHZXngM_Ux2sO27_GEgKpZsY8DMMsaYI4dRSIrk3eMNViT7tHNCkBjQn2ywSMnt4rCPCR22O_hmYesFuBVyi0MDWpKw?testcase_id=4645927739719680

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 12 by ClusterFuzz, Jan 12 2017

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

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

Comment 13 by sheriffbot@chromium.org, Jan 12 2017

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

Comment 14 by sheriffbot@chromium.org, Apr 20 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