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

Issue 699937 link

Starred by 2 users

Issue metadata

Status: Verified
Owner: ----
Closed: Apr 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

Integer-overflow in SkIRect::width

Project Member Reported by ClusterFuzz, Mar 9 2017

Issue description

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

Fuzzer: ifratric-browserfuzzer-v3
Job Type: linux_ubsan_chrome
Platform Id: linux

Crash Type: Integer-overflow
Crash Address: 
Crash State:
  SkIRect::width
  size
  SkMatrixImageFilter::onFilterImage
  
Sanitizer: undefined (UBSAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=420630:420810

Reproducer Testcase: https://clusterfuzz.com/download/AMIfv96bQ6VoLrX9t_Ov_h72rT47LfoTqIEGQm8U9R99A52uCsnRx50OWakawqKp9CgepvPh3x5Fyfxb8vCz62E3G68e4q-8Rf8snZM6rMANSec1ajwZ2As4YUvUAR9r9KXX-2DQlD_F587rB693kI2o8DBWu095rw?testcase_id=6385970841911296


Issue filed automatically.

See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs for more information.
 
Cc: reed@chromium.org mtklein@chromium.org brianosman@chromium.org
Components: Internals>Skia
Labels: Test-Predator-Wrong M-59
The result is a list of CLs that change the crashed files. 

Author: brianosman
Project: chromium-skia
Changelist: https://chromium.googlesource.com/skia.git/+/53c38087949252d27cde668368a3eeb59cc2eb00
Time: Fri Sep 23 15:11:55 2016
Lines 73 of file SkMatrixImageFilter.cpp which potentially caused crash are changed in this cl (frame #2, "SkMatrixImageFilter::onFilterImage").

could someone please take a look?.
Thank you

Comment 2 by hcm@chromium.org, Apr 18 2017

Labels: -Pri-2 Pri-3
Lowering priority on non-security integer overflows- while unexpected rendering behavior is not ideal, it is also not work that can be prioritized at this time.
Project Member

Comment 3 by ClusterFuzz, Apr 20 2017

ClusterFuzz has detected this issue as fixed in range 465517:465532.

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

Fuzzer: ifratric-browserfuzzer-v3
Job Type: linux_ubsan_chrome
Platform Id: linux

Crash Type: Integer-overflow
Crash Address: 
Crash State:
  SkIRect::width
  size
  SkMatrixImageFilter::onFilterImage
  
Sanitizer: undefined (UBSAN)

Regressed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=420630:420810
Fixed: https://clusterfuzz.com/revisions?job=linux_ubsan_chrome&range=465517:465532

Reproducer Testcase: https://clusterfuzz.com/download/AMIfv96bQ6VoLrX9t_Ov_h72rT47LfoTqIEGQm8U9R99A52uCsnRx50OWakawqKp9CgepvPh3x5Fyfxb8vCz62E3G68e4q-8Rf8snZM6rMANSec1ajwZ2As4YUvUAR9r9KXX-2DQlD_F587rB693kI2o8DBWu095rw?testcase_id=6385970841911296


See https://dev.chromium.org/Home/chromium-security/bugs/reproducing-clusterfuzz-bugs 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 4 by ClusterFuzz, Apr 20 2017

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

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

Sign in to add a comment