Re: [Ffmpeg-cvslog] r7537 - trunk/libavformat/avidec.c

Top Page

Reply to this message
Author: Guillaume POIRIER
Date:  
To: FFmpeg Subversion commit log and discussion
Subject: Re: [Ffmpeg-cvslog] r7537 - trunk/libavformat/avidec.c
Hi,

On 1/15/07, michael <subversion@???> wrote:

> fixing segfault with http://sam.zoy.org/zzuf/lol-ffplay.avi


> --- trunk/libavformat/avidec.c (original)
> +++ trunk/libavformat/avidec.c Mon Jan 15 23:56:46 2007
> @@ -397,7 +397,7 @@
> break;
> case MKTAG('s', 't', 'r', 'f'):
> /* stream header */
> - if (stream_index >= s->nb_streams || avi->dv_demux) {
> + if (stream_index >= (unsigned)s->nb_streams || avi->dv_demux) {
> url_fskip(pb, size);
> } else {


That's interesting: why this fix instead of defining nb_streams struct
member as unsigned int?

Wouldn't that be more logical?

Guillaume
_______________________________________________
ffmpeg-cvslog mailing list
ffmpeg-cvslog@???
http://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-cvslog