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

Issue 694599 link

Starred by 2 users

Issue metadata

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



Sign in to add a comment

Direct-leak in OJPEGReadHeaderInfoSecStreamDht

Project Member Reported by ClusterFuzz, Feb 21 2017

Issue description

Cc: msrchandra@chromium.org
Labels: Test-Predator-Wrong-CLs M-57
Owner: npm@chromium.org
Status: Assigned (was: Untriaged)
Predator and CL did not find any possible suspects.
Using Code Search for the file, "tif_ojpeg.c" assigning to the concern owner.
Suspecting Commit#
https://pdfium.git.corp.google.com/pdfium.git/+/73debd4d226114b88430f2cc30dac056be5c13f3#

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

Comment 2 by ta...@google.com, Feb 24 2017

We have a command-line tool to reproduce this crash locally (on Goobuntu) from chromium source code with a single command. It'd really help speed up the development process. The tool is in alpha. Ping tanin@ or everestmz@ if you are interested.

Comment 3 by npm@chromium.org, Feb 24 2017

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

Comment 4 by bugdroid1@chromium.org, Feb 24 2017

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

commit b8a9a5a4dc6fddf60fb247db3fca17a42662632a
Author: pdfium-deps-roller <pdfium-deps-roller@chromium.org>
Date: Fri Feb 24 23:38:15 2017

Roll src/third_party/pdfium/ 37e2bd1ac..412fa6533 (2 commits).

https://pdfium.googlesource.com/pdfium.git/+log/37e2bd1acf84..412fa65331cc

$ git log 37e2bd1ac..412fa6533 --date=short --no-merges --format='%ad %ae %s'
2017-02-24 npm Remove repeated flags from CPDF_Font
2017-02-23 npm Libtiff: fix leaking tables in tif_ojpeg.c

Created with:
  roll-dep src/third_party/pdfium
BUG= 694599 

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/2717533006
Cr-Commit-Position: refs/heads/master@{#452998}

[modify] https://crrev.com/b8a9a5a4dc6fddf60fb247db3fca17a42662632a/DEPS

Components: Internals>Plugins>PDF
Labels: -M-57 M-58
It's a memory leak and not a regression. We probably don't need this for M57.
Status: Fixed (was: Started)

Sign in to add a comment