Re: [libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-16 Thread Luca Barbato
On 11/16/11 9:25 AM, Mike Melanson wrote: On 11/16/2011 12:22 AM, Johan Andersson wrote: - * after the frame header. Else, assume 2 extra bytes. */ + * after the frame header. Else, assume 2 extra bytes. The container size + * size also cannot be a multiple of the encoded size. */ "The contain

Re: [libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-16 Thread Martin Storsjö
On Wed, 16 Nov 2011, Kostya Shishkov wrote: On Tue, Nov 15, 2011 at 11:06:48PM -0800, m...@multimedia.cx wrote: From: Mike Melanson http://samples.mplayerhq.hu/V-codecs/CVID/bad_cinepak_frame_size.mov This fix works around another work around which handles a different type of odd Cinepak dat

Re: [libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-16 Thread Kostya Shishkov
On Wed, Nov 16, 2011 at 12:25:33AM -0800, Mike Melanson wrote: > On 11/16/2011 12:22 AM, Johan Andersson wrote: > >>- * after the frame header. Else, assume 2 extra bytes. */ > >>+ * after the frame header. Else, assume 2 extra bytes. The > >>container size > >>+

Re: [libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-16 Thread Mike Melanson
On 11/16/2011 12:25 AM, Kostya Shishkov wrote: On Wed, Nov 16, 2011 at 09:22:02AM +0100, Johan Andersson wrote: On Tue, Nov 15, 2011 at 11:06:48PM -0800, m...@multimedia.cx wrote: From: Mike Melanson http://samples.mplayerhq.hu/V-codecs/CVID/bad_cinepak_frame_size.mov This fix works around an

Re: [libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-16 Thread Mike Melanson
On 11/16/2011 12:22 AM, Johan Andersson wrote: - * after the frame header. Else, assume 2 extra bytes. */ + * after the frame header. Else, assume 2 extra bytes. The container size + * size also cannot be a multiple of the encoded size. */ "The container siz

Re: [libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-16 Thread Kostya Shishkov
On Wed, Nov 16, 2011 at 09:22:02AM +0100, Johan Andersson wrote: > On Tue, Nov 15, 2011 at 11:06:48PM -0800, m...@multimedia.cx wrote: > > From: Mike Melanson > > > > http://samples.mplayerhq.hu/V-codecs/CVID/bad_cinepak_frame_size.mov > > > > This fix works around another work around which hand

Re: [libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-16 Thread Johan Andersson
On Tue, Nov 15, 2011 at 11:06:48PM -0800, m...@multimedia.cx wrote: > From: Mike Melanson > > http://samples.mplayerhq.hu/V-codecs/CVID/bad_cinepak_frame_size.mov > > This fix works around another work around which handles a different type > of odd Cinepak data. > > Thanks to Matthew Hoops (clo

Re: [libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-16 Thread Kostya Shishkov
On Tue, Nov 15, 2011 at 11:06:48PM -0800, m...@multimedia.cx wrote: > From: Mike Melanson > > http://samples.mplayerhq.hu/V-codecs/CVID/bad_cinepak_frame_size.mov > > This fix works around another work around which handles a different type > of odd Cinepak data. > > Thanks to Matthew Hoops (clo

[libav-devel] [PATCH 2/2] Add another special case to the Cinepak video decoder so that it can handle the following file:

2011-11-15 Thread mike
From: Mike Melanson http://samples.mplayerhq.hu/V-codecs/CVID/bad_cinepak_frame_size.mov This fix works around another work around which handles a different type of odd Cinepak data. Thanks to Matthew Hoops (clone2727 - gmail.com) for the sample and fix. --- libavcodec/cinepak.c |5 +++--