Chromium Code Reviews
Help | Chromium Project | Sign in
(845)

Issue 11575026: Fix possible integer overflow in Opus with ~16MB fed to the decoder. (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 4 months ago by aedla
Modified:
1 year, 4 months ago
Reviewers:
Sergey Ulanov
CC:
chromium-reviews_chromium.org
Visibility:
Public.

Description

Fix possible integer overflow in Opus with ~16MB fed to the decoder.

BUG=160480
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=173498

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fix possible integer overflow in Opus with ~16MB fed to the decoder. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -3 lines) Lint Patch
M src/opus_decoder.c View 1 1 chunk +1 line, -3 lines 0 comments 1 errors Download
Trybot results:
Commit:

Messages

Total messages: 7
aedla
Importing a fix from upstream. Sergey, will you please take a look?
1 year, 4 months ago #1
Sergey Ulanov
Changes in opus_decoder.c - LGTM. I think README.chromium should be updated in a separate CL, ...
1 year, 4 months ago #2
aedla
Fix possible integer overflow in Opus with ~16MB fed to the decoder. BUG=160480
1 year, 4 months ago #3
aedla
On 2012/12/14 18:51:22, sergeyu wrote: > Changes in opus_decoder.c - LGTM. I think README.chromium should ...
1 year, 4 months ago #4
Sergey Ulanov
On 2012/12/16 10:39:57, aedla wrote: > On 2012/12/14 18:51:22, sergeyu wrote: > > Changes in ...
1 year, 4 months ago #5
Sergey Ulanov
Ah, CQ doesn't work for /trunk/deps. I'll land this change.
1 year, 4 months ago #6
Sergey Ulanov
1 year, 4 months ago #7
Message was sent while issue was closed.
On 2012/12/17 18:26:10, sergeyu wrote:
> Ah, CQ doesn't work for /trunk/deps. I'll land this change.

Landed in crrev.com/173498
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1280:2d3e6564b7b6