Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(385)

Issue 7137002: Don't forget the ffmpeg input buffer padding when allocating a codec's (Closed)

Created:
9 years, 6 months ago by Chris Evans
Modified:
9 years, 6 months ago
CC:
chromium-reviews, hclam+watch_chromium.org, sjl, ddorwin+watch_chromium.org, fischman+watch_chromium.org, acolwell GONE FROM CHROMIUM, annacc, ajwong+watch_chromium.org, vrk (LEFT CHROMIUM), scherkus (not reviewing)
Visibility:
Public.

Description

Don't forget the ffmpeg input buffer padding when allocating a codec's extradata buffer. BUG=82438 Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=88354

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -2 lines) Patch
M media/video/ffmpeg_video_decode_engine.cc View 1 chunk +4 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Chris Evans
For whoever sees it first...
9 years, 6 months ago (2011-06-08 07:56:58 UTC) #1
fbarchard
On 2011/06/08 07:56:58, Chris Evans wrote: > For whoever sees it first... LGTM
9 years, 6 months ago (2011-06-08 08:19:51 UTC) #2
scherkus (not reviewing)
9 years, 6 months ago (2011-06-08 21:16:06 UTC) #3
LGTM

nice catch!
On Jun 8, 2011 1:19 AM, <fbarchard@chromium.org> wrote:
> On 2011/06/08 07:56:58, Chris Evans wrote:
>> For whoever sees it first...
>
> LGTM
>
> http://codereview.chromium.org/7137002/

Powered by Google App Engine
This is Rietveld 408576698