[FFmpeg-devel] [PATCH 4/4] EA cdata demuxer: support files with 20 byte header

[Peter Ross]

On Sat, Apr 23, 2011 at 05:44:45PM +0200, Reimar Döffinger wrote:
> On Sat, Apr 23, 2011 at 10:09:19PM +1000, Peter Ross wrote:
> > ---
> >  libavformat/eacdata.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/libavformat/eacdata.c b/libavformat/eacdata.c
> > index 1a9f3d0..cc1f833 100644
> > --- a/libavformat/eacdata.c
> > +++ b/libavformat/eacdata.c
> > @@ -63,7 +63,7 @@ static int cdata_read_header(AVFormatContext *s, AVFormatParameters *ap)
> >      };
> >  
> >      sample_rate = avio_rb16(pb);
> > -    avio_skip(pb, 12);
> > +    avio_skip(pb, (avio_r8(pb) & 0x20) ? 15 : 11);
> 
> Any idea what is in those bytes?
> Might allow to write this in a way that "makes more sense",

Three of the four bytes are a repeat of what is found at file offset 0x0005 in the file.

> but in principle it's obviously ok as-is if it works.

It works.

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110424/627fd2c9/attachment.asc>