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

Issue 648069 link

Starred by 1 user

Issue metadata

Status: Duplicate
Merged: issue 646347
Owner:
Closed: Oct 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Security



Sign in to add a comment

Negative-size-param in sfntly::MemoryByteArray::InternalGet

Project Member Reported by ClusterFuzz, Sep 18 2016

Issue description

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

Fuzzer: libfuzzer_sfntly_fuzzer
Job Type: libfuzzer_chrome_asan
Platform Id: linux

Crash Type: Negative-size-param
Crash Address: 
Crash State:
  sfntly::MemoryByteArray::InternalGet
  sfntly::NameTable::Name
  HasName
  

Minimized Testcase (1.03 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94NsvmNaxdP1C38rVMFSaid0Li5PVjkmSq5AJElMzgw05ENpk0JnVLDyC66989FkOBx3T0rY72b0IAJYmvbHNNo0KyBYNSfZzrKf1vIBsYwnmy6ALMqn2Up--cogget4hxOgSEIdw36772YwMg1bzTxO0PaKA?testcase_id=5065060257103872

Issue manually filed by: mmoroz

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

Comment 1 by mmoroz@chromium.org, Sep 18 2016

Cc: mmoroz@chromium.org kcc@chromium.org aizatsky@chromium.org
Owner: thestig@chromium.org
thestig@, assigning this to you as an author of the fuzzer + one of the OWNERS for https://cs.chromium.org/chromium/src/third_party/sfntly/OWNERS
Project Member

Comment 2 by sheriffbot@chromium.org, Sep 19 2016

Labels: Pri-1
Project Member

Comment 3 by sheriffbot@chromium.org, Sep 19 2016

Status: Assigned (was: Untriaged)
Components: Internals>Printing
Cc: behdad@chromium.org
Components: -Internals>Printing Internals>Skia>PDF
Labels: Security_Impact-Head
Project Member

Comment 7 by sheriffbot@chromium.org, Sep 24 2016

Labels: M-55
Project Member

Comment 8 by sheriffbot@chromium.org, Sep 24 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

Comment 9 by gov...@chromium.org, Sep 29 2016

This bug is reported as M55 Beta blocker.Please try to resolve this before M55 branch on Oct 6th,2016 so it has enough baking time in Dev.


Cc: kenrb@chromium.org thestig@chromium.org
 Issue 652012  has been merged into this issue.
A friendly reminder that M55 Beta launch is coming soon! Your bug is labelled as Beta ReleaseBlock, pls make sure to land the fix ASAP. Thank you.

**** Bulk edit -  please ignore if not applicable ****

This bug  is reported as M55 Beta blocker and we're getting closer to M55 Beta promotion. 
Please plan to have fix ready and merged to M55 branch (2883) by 5:00 PM PT, Monday(10/10) so it has enough baking time in Dev before Beta promotion. Thank you.
Project Member

Comment 13 by sheriffbot@chromium.org, Oct 13 2016

Labels: -Security_Impact-Head Security_Impact-Beta
Labels: -ReleaseBlock-Beta ReleaseBlock-Stable
Changing to release block stable due to suggestion in 652012 that this isn't a regression.
Project Member

Comment 15 by sheriffbot@chromium.org, Oct 15 2016

thestig: Uh oh! This issue still open and hasn't been updated in the last 14 days. This is a serious vulnerability, and we want to ensure that there's progress. Could you please leave an update with the current status and any potential blockers?

If you're not the right owner for this issue, could you please remove yourself as soon as possible or help us find the right one?

If the issue is fixed or you can't reproduce it, please close the bug. If you've started working on a fix, please set the status to Started.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
The  bug 646347  fix takes care of this as well.
Mergedinto: 646347
Status: Duplicate (was: Assigned)
Project Member

Comment 18 by ClusterFuzz, Oct 20 2016

ClusterFuzz has detected this issue as fixed in range 426270:426317.

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

Fuzzer: libfuzzer_sfntly_fuzzer
Job Type: libfuzzer_chrome_asan
Platform Id: linux

Crash Type: Negative-size-param
Crash Address: 
Crash State:
  sfntly::MemoryByteArray::InternalGet
  sfntly::NameTable::Name
  HasName
  
Regressed: https://cluster-fuzz.appspot.com/revisions?job=libfuzzer_chrome_asan&range=417024:417277
Fixed: https://cluster-fuzz.appspot.com/revisions?job=libfuzzer_chrome_asan&range=426270:426317

Minimized Testcase (1.03 Kb): https://cluster-fuzz.appspot.com/download/AMIfv94NsvmNaxdP1C38rVMFSaid0Li5PVjkmSq5AJElMzgw05ENpk0JnVLDyC66989FkOBx3T0rY72b0IAJYmvbHNNo0KyBYNSfZzrKf1vIBsYwnmy6ALMqn2Up--cogget4hxOgSEIdw36772YwMg1bzTxO0PaKA?testcase_id=5065060257103872

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 19 by sheriffbot@chromium.org, Jan 26 2017

Labels: -Restrict-View-SecurityTeam 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