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

Issue 680102 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Direct-leak in opj_j2k_read_mct

Project Member Reported by ClusterFuzz, Jan 11 2017

Issue description

Cc: msrchandra@chromium.org
Labels: Test-Predator-Correct-CLs
Owner: npm@chromium.org
Status: Assigned (was: Untriaged)
Assigning to the concern owner from Find it results --
The result is a list of CLs that change the crashed files. 

Author: Nicolas Pena
Project: chromium-pdfium
Changelist: https://pdfium.googlesource.com/pdfium.git/+/44bc1f818dd791c2a5a81103be3853093fd934b3
Time: Tue Jan 10 16:40:29 2017 -0500
Lines 5224-5226 of file j2k.c which potentially caused crash are changed in this cl (frame #1, "opj_j2k_read_mct").
Minimum distance from crash line to modified line: 0. (file: j2k.c, crashed on: 5224, modified: 5224).

@Nicolas Pena -- Could you please look into the issue, kindly re-assign if this is not related to your changes.
Thank You.

Comment 2 by npm@chromium.org, Jan 11 2017

Status: Started (was: Assigned)
https://pdfium-review.googlesource.com/c/2171/
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 11 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/68e998adb1ea9fd998afd4dafd372d3d4729a54b

commit 68e998adb1ea9fd998afd4dafd372d3d4729a54b
Author: pdfium-deps-roller <pdfium-deps-roller@chromium.org>
Date: Wed Jan 11 23:46:19 2017

Roll src/third_party/pdfium/ 8fa82794f..96f482c9c (2 commits).

https://pdfium.googlesource.com/pdfium.git/+log/8fa82794ffc2..96f482c9cd3c

$ git log 8fa82794f..96f482c9c --date=short --no-merges --format='%ad %ae %s'
2017-01-11 dsinclair Convert FDE CSS enums to enum classes.
2017-01-11 npm Really fix m_nb_mct_records calculation in opj_j2k_read_mct

BUG= 678461 , 680102 

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls

TBR=dsinclair@chromium.org

Review-Url: https://codereview.chromium.org/2623103004
Cr-Commit-Position: refs/heads/master@{#443057}

[modify] https://crrev.com/68e998adb1ea9fd998afd4dafd372d3d4729a54b/DEPS

Project Member

Comment 5 by ClusterFuzz, Jan 12 2017

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

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
Components: Internals>Plugins>PDF

Sign in to add a comment