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

Issue 689339 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit > 30 days ago
Closed: Feb 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug-Regression

Blocking:
issue 777555



Sign in to add a comment

Direct-leak in av_reallocp

Project Member Reported by ClusterFuzz, Feb 7 2017

Issue description

Cc: nyerramilli@chromium.org ddorwin@chromium.org
Labels: -Type-Bug Test-Predator-Wrong-CLs M-58 Type-Bug-Regression
Owner: dalecur...@chromium.org
Status: Assigned (was: Untriaged)
Findit cannot find any culprit results.

assigning to /src/third_party/ffmpeg/OWNERS, request to check the issue and help.
Cc: liber...@chromium.org
Owner: hubbe@chromium.org
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 7 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/third_party/ffmpeg/+/a6a79bda55868f7faee0f183a45191d3251fb5f1

commit a6a79bda55868f7faee0f183a45191d3251fb5f1
Author: Fredrik Hubinette <hubbe@google.com>
Date: Tue Feb 07 20:30:51 2017

Fix minor ffmpeg memory leak in id3v2 parsing.

BUG= 689339 

Change-Id: Id17b3458f469da72986e2f5090f25118be2c61d4
Reviewed-on: https://chromium-review.googlesource.com/439405
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>

[modify] https://crrev.com/a6a79bda55868f7faee0f183a45191d3251fb5f1/libavformat/id3v2.c
[modify] https://crrev.com/a6a79bda55868f7faee0f183a45191d3251fb5f1/chromium/patches/README

Project Member

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

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

commit b4dcc0a8ef699fa581b78b9a7912a23d45e5ac5b
Author: wolenetz <wolenetz@chromium.org>
Date: Wed Feb 08 00:57:52 2017

Roll src/third_party/ffmpeg/ 785631123..239c9f9e2 (2 commits).

https://chromium.googlesource.com/chromium/third_party/ffmpeg.git/+log/785631123a41..239c9f9e2754

$ git log 785631123..239c9f9e2 --date=short --no-merges --format='%ad %ae %s'
2017-02-07 wolenetz avcodec/flacdsp: Avoid undefined operations in non debug builds
2017-02-07 hubbe Fix minor ffmpeg memory leak in id3v2 parsing.

BUG=686513,640820, 689339 
TBR=hubbe@chromium.org,dalecurtis@chromium.org

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

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

Project Member

Comment 6 by ClusterFuzz, Feb 8 2017

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

If this is incorrect, please add ClusterFuzz-Wrong label and re-open the issue.
hubbe@, did you attempt to upstream this?

Comment 8 by hubbe@chromium.org, Oct 23 2017

Don't remember. (Which probably means no.)


Can you go ahead and submit it upstream then? Upstreaming patches is each roller's responsibility.
Components: Internals>Media
Blocking: 777555

Sign in to add a comment