'code' is used to index a table so make it unsigned
authorAurelien Jacobs <aurel@gnuage.org>
Wed, 21 Jan 2009 22:32:17 +0000 (22:32 +0000)
committerAurelien Jacobs <aurel@gnuage.org>
Wed, 21 Jan 2009 22:32:17 +0000 (22:32 +0000)
this prevents using negative index and possibly crashing with broken files

Originally committed as revision 16714 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavformat/isom.c
libavformat/isom.h
libavformat/mov.c

index 73ee328..39c58c6 100644 (file)
@@ -274,7 +274,7 @@ int ff_mov_iso639_to_lang(const char *lang, int mp4)
     return code;
 }
 
-int ff_mov_lang_to_iso639(int code, char *to)
+int ff_mov_lang_to_iso639(unsigned code, char *to)
 {
     int i;
     /* is it the mangled iso code? */
index 934dde8..2cad5b8 100644 (file)
@@ -33,7 +33,7 @@ extern const AVCodecTag codec_movaudio_tags[];
 extern const AVCodecTag ff_codec_movsubtitle_tags[];
 
 int ff_mov_iso639_to_lang(const char *lang, int mp4);
-int ff_mov_lang_to_iso639(int code, char *to);
+int ff_mov_lang_to_iso639(unsigned code, char *to);
 
 typedef struct {
     int count;
index fc946ea..0a93444 100644 (file)
@@ -494,7 +494,7 @@ static int mov_read_mdhd(MOVContext *c, ByteIOContext *pb, MOVAtom atom)
     AVStream *st = c->fc->streams[c->fc->nb_streams-1];
     MOVStreamContext *sc = st->priv_data;
     int version = get_byte(pb);
-    int lang;
+    unsigned lang;
 
     if (version > 1)
         return -1; /* unsupported */