au: return AVERROR codes instead of -1
authorJustin Ruggles <justin.ruggles@gmail.com>
Sun, 23 Dec 2012 20:06:46 +0000 (15:06 -0500)
committerJustin Ruggles <justin.ruggles@gmail.com>
Wed, 9 Jan 2013 16:52:57 +0000 (11:52 -0500)
libavformat/au.c

index d88ccfc..488e2b0 100644 (file)
@@ -69,7 +69,7 @@ static int au_read_header(AVFormatContext *s)
 
     tag = avio_rl32(pb);
     if (tag != MKTAG('.', 's', 'n', 'd'))
 
     tag = avio_rl32(pb);
     if (tag != MKTAG('.', 's', 'n', 'd'))
-        return -1;
+        return AVERROR_INVALIDDATA;
     size = avio_rb32(pb); /* header size */
     avio_rb32(pb);        /* data size */
 
     size = avio_rb32(pb); /* header size */
     avio_rb32(pb);        /* data size */
 
@@ -107,7 +107,7 @@ static int au_read_header(AVFormatContext *s)
 
     st = avformat_new_stream(s, NULL);
     if (!st)
 
     st = avformat_new_stream(s, NULL);
     if (!st)
-        return -1;
+        return AVERROR(ENOMEM);
     st->codec->codec_type  = AVMEDIA_TYPE_AUDIO;
     st->codec->codec_tag   = id;
     st->codec->codec_id    = codec;
     st->codec->codec_type  = AVMEDIA_TYPE_AUDIO;
     st->codec->codec_tag   = id;
     st->codec->codec_id    = codec;
@@ -160,7 +160,7 @@ AVInputFormat ff_au_demuxer = {
 static int put_au_header(AVIOContext *pb, AVCodecContext *enc)
 {
     if (!enc->codec_tag)
 static int put_au_header(AVIOContext *pb, AVCodecContext *enc)
 {
     if (!enc->codec_tag)
-        return -1;
+        return AVERROR(EINVAL);
 
     ffio_wfourcc(pb, ".snd");                   /* magic number */
     avio_wb32(pb, 24);                          /* header size */
 
     ffio_wfourcc(pb, ".snd");                   /* magic number */
     avio_wb32(pb, 24);                          /* header size */
@@ -175,11 +175,12 @@ static int put_au_header(AVIOContext *pb, AVCodecContext *enc)
 static int au_write_header(AVFormatContext *s)
 {
     AVIOContext *pb = s->pb;
 static int au_write_header(AVFormatContext *s)
 {
     AVIOContext *pb = s->pb;
+    int ret;
 
     s->priv_data = NULL;
 
 
     s->priv_data = NULL;
 
-    if (put_au_header(pb, s->streams[0]->codec) < 0)
-        return -1;
+    if ((ret = put_au_header(pb, s->streams[0]->codec)) < 0)
+        return ret;
 
     avio_flush(pb);
 
 
     avio_flush(pb);