[FFmpeg-devel] [PATCH] alsdec: error out if channel number changes

Thilo Borgmann thilo.borgmann at mail.de
Mon Jan 4 21:18:35 CET 2016


Am 04.01.16 um 18:31 schrieb Derek Buitenhuis:
> On 1/4/2016 3:18 PM, Andreas Cadhalpun wrote:
>> Supporting this would require re-initialization to change buffer sizes.
> 
> I may be mistaken, but don't we already support some codecs which do
> this, properly?

I'm quite sure that changing the number of channels is invalid for ALS
so there is no sense in implementing it.

However, we should should add a fix for out-of-bound reads. An updated
patch giving a proper error would do, I think.

-Thilo




More information about the ffmpeg-devel mailing list