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

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

[aedla, 2012-12-14 08:07:05]

Importing a fix from upstream. Sergey, will you please take a look?

[Sergey Ulanov, 2012-12-14 18:51:22]

Changes in opus_decoder.c - LGTM. I think README.chromium should be updated in a
separate CL, together with DEPS roll.

https://codereview.chromium.org/11575026/diff/1/src/third_party/opus/README.c...
File src/third_party/opus/README.chromium (right):

https://codereview.chromium.org/11575026/diff/1/src/third_party/opus/README.c...
src/third_party/opus/README.chromium:11: Modifications:
The change in opus_decoder.c will need a DEPS roll. It's better to land it first
and then update README.chromium in the same change where you roll opus in DEPS.

[aedla, 2012-12-16 10:21:42]

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

BUG=160480

[aedla, 2012-12-16 10:39:57]

On 2012/12/14 18:51:22, sergeyu wrote:
> Changes in opus_decoder.c - LGTM. I think README.chromium should be updated in
a
> separate CL, together with DEPS roll.
> 
>
https://codereview.chromium.org/11575026/diff/1/src/third_party/opus/README.c...
> File src/third_party/opus/README.chromium (right):
> 
>
https://codereview.chromium.org/11575026/diff/1/src/third_party/opus/README.c...
> src/third_party/opus/README.chromium:11: Modifications:
> The change in opus_decoder.c will need a DEPS roll. It's better to land it
first
> and then update README.chromium in the same change where you roll opus in
DEPS.

Got it I think. Does a DEPS roll mean an update of the revision number in
src/DEPS?

[Sergey Ulanov, 2012-12-17 05:25:00]

On 2012/12/16 10:39:57, aedla wrote:
> On 2012/12/14 18:51:22, sergeyu wrote:
> > Changes in opus_decoder.c - LGTM. I think README.chromium should be updated
in
> a
> > separate CL, together with DEPS roll.
> > 
> >
>
https://codereview.chromium.org/11575026/diff/1/src/third_party/opus/README.c...
> > File src/third_party/opus/README.chromium (right):
> > 
> >
>
https://codereview.chromium.org/11575026/diff/1/src/third_party/opus/README.c...
> > src/third_party/opus/README.chromium:11: Modifications:
> > The change in opus_decoder.c will need a DEPS roll. It's better to land it
> first
> > and then update README.chromium in the same change where you roll opus in
> DEPS.
> 
> Got it I think. Does a DEPS roll mean an update of the revision number in
> src/DEPS?

That's right.

[Sergey Ulanov, 2012-12-17 18:26:10]

Ah, CQ doesn't work for /trunk/deps. I'll land this change.

[Sergey Ulanov, 2012-12-17 18:45:53]

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