Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(23)

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

Created:
5 years, 1 month ago by aedla
Modified:
5 years, 1 month ago
Reviewers:
Sergey Ulanov
CC:
chromium-reviews
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) Patch
M src/opus_decoder.c View 1 1 chunk +1 line, -3 lines 0 comments Download

Messages

Total messages: 7 (0 generated)
aedla
Importing a fix from upstream. Sergey, will you please take a look?
5 years, 1 month ago (2012-12-14 08:07:05 UTC) #1
Sergey Ulanov
Changes in opus_decoder.c - LGTM. I think README.chromium should be updated in a separate CL, ...
5 years, 1 month ago (2012-12-14 18:51:22 UTC) #2
aedla
Fix possible integer overflow in Opus with ~16MB fed to the decoder. BUG=160480
5 years, 1 month ago (2012-12-16 10:21:42 UTC) #3
aedla
On 2012/12/14 18:51:22, sergeyu wrote: > Changes in opus_decoder.c - LGTM. I think README.chromium should ...
5 years, 1 month ago (2012-12-16 10:39:57 UTC) #4
Sergey Ulanov
On 2012/12/16 10:39:57, aedla wrote: > On 2012/12/14 18:51:22, sergeyu wrote: > > Changes in ...
5 years, 1 month ago (2012-12-17 05:25:00 UTC) #5
Sergey Ulanov
Ah, CQ doesn't work for /trunk/deps. I'll land this change.
5 years, 1 month ago (2012-12-17 18:26:10 UTC) #6
Sergey Ulanov
5 years, 1 month ago (2012-12-17 18:45:53 UTC) #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

Powered by Google App Engine
This is Rietveld 408576698