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

Guillaume POIRIER poirierg
Tue Jan 16 00:12:15 CET 2007


Hi,

On 1/15/07, michael <subversion at mplayerhq.hu> 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




More information about the ffmpeg-cvslog mailing list