[FFmpeg-devel] [PATCH] oggparseskeleton: Fix fisbone header parsing

Michael Niedermayer michaelni at gmx.at
Sat Jul 6 21:12:15 CEST 2013


On Sat, Jul 06, 2013 at 01:38:32PM -0300, James Almer wrote:
> On 06/07/13 8:37 AM, Michael Niedermayer wrote:
> >> diff --git a/libavformat/oggparseskeleton.c b/libavformat/oggparseskeleton.c
> >> index 307387d..56f5713 100644
> >> --- a/libavformat/oggparseskeleton.c
> >> +++ b/libavformat/oggparseskeleton.c
> > 
> >> @@ -37,9 +37,6 @@ static int skeleton_header(AVFormatContext *s, int idx)
> >>      strcpy(st->codec->codec_name, "skeleton");
> >>      st->codec->codec_type = AVMEDIA_TYPE_DATA;
> >>  
> >> -    if (os->psize < 8)
> >> -        return -1;
> >> -
> >>      if (!strncmp(buf, "fishead", 8)) {
> >>          if (os->psize < 64)
> >>              return -1;
> > 
> > why?
> > this is supposed to check if buf contains 8 bytes for the strncmp
> > 
> > rest of the patch split & applied
> > 
> > thanks
> 
> That line generates an error about the Skeleton header parsing failing.
> This is because the EOS page has a single packet of size 0, so skeleton_header()
> returns -1.

please upload that ogg file

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- 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/20130706/77bf555b/attachment.asc>


More information about the ffmpeg-devel mailing list