Re: [FFmpeg-devel] [PATCH] ffm: reject a negative codec_id

2015-12-15 Thread Andreas Cadhalpun
On 15.12.2015 03:15, Michael Niedermayer wrote: > On Tue, Dec 15, 2015 at 01:19:01AM +0100, Andreas Cadhalpun wrote: >> On 15.12.2015 00:43, Michael Niedermayer wrote: >> ffmdec.c | 28 >> 1 file changed, 28 insertions(+) >> 5a300bbcfb78b1f24b8b7c572d420577b2a1b1f3

Re: [FFmpeg-devel] [PATCH] ffm: reject a negative codec_id

2015-12-14 Thread Michael Niedermayer
On Tue, Dec 15, 2015 at 12:03:22AM +0100, Andreas Cadhalpun wrote: > A negative codec_id cannot be handled by the found_decoder API of > AVStream->info: if the codec_id is not recognized, found_decoder is set > to -codec_id, which has to be '<0' according to the API documentation. > > This can

Re: [FFmpeg-devel] [PATCH] ffm: reject a negative codec_id

2015-12-14 Thread Andreas Cadhalpun
On 15.12.2015 00:43, Michael Niedermayer wrote: > On Tue, Dec 15, 2015 at 12:03:22AM +0100, Andreas Cadhalpun wrote: >> A negative codec_id cannot be handled by the found_decoder API of >> AVStream->info: if the codec_id is not recognized, found_decoder is set >> to -codec_id, which has to be '<0'

Re: [FFmpeg-devel] [PATCH] ffm: reject a negative codec_id

2015-12-14 Thread Michael Niedermayer
On Tue, Dec 15, 2015 at 01:19:01AM +0100, Andreas Cadhalpun wrote: > On 15.12.2015 00:43, Michael Niedermayer wrote: > > On Tue, Dec 15, 2015 at 12:03:22AM +0100, Andreas Cadhalpun wrote: > >> A negative codec_id cannot be handled by the found_decoder API of > >> AVStream->info: if the codec_id is