Re: [FFmpeg-devel] [PATCH] lavc/libopusdec.c Fix ff_vorbis_channel_layouts OOB

2016-12-15 Thread Matthew Wolenetz
Ah, you're right. My fix was based on a slightly earlier version that didn't yet have your fix in it. Disregard my patch. Matt On Wed, Dec 14, 2016 at 5:43 PM, Andreas Cadhalpun < andreas.cadhal...@googlemail.com> wrote: > On 15.12.2016 00:39, Matthew Wolenetz wrote: > > From 141e56ccf7fc5664642

Re: [FFmpeg-devel] [PATCH] lavc/libopusdec.c Fix ff_vorbis_channel_layouts OOB

2016-12-14 Thread Andreas Cadhalpun
On 15.12.2016 00:39, Matthew Wolenetz wrote: > From 141e56ccf7fc56646424484d357b6c74a486d2e2 Mon Sep 17 00:00:00 2001 > From: Matt Wolenetz > Date: Mon, 21 Nov 2016 17:30:50 -0800 > Subject: [PATCH] lavc/libopusdec.c Fix ff_vorbis_channel_layouts OOB > > Similar to existing lavc/vorbisdec.c code

[FFmpeg-devel] [PATCH] lavc/libopusdec.c Fix ff_vorbis_channel_layouts OOB

2016-12-14 Thread Matthew Wolenetz
Similar to existing lavc/vorbisdec.c code which first checks that avc->channels is valid for accessing ff_vorbis_channel_layouts, this change adds protection to libopusdec.c to prevent accessing that array with a negative index. Reference https://crbug.com/666794. From 141e56ccf7fc56646424484d357b6