lavc/libvorbisdec: use better error codes
authorLukasz Marek <lukasz.m.luki2@gmail.com>
Sat, 22 Nov 2014 21:45:10 +0000 (22:45 +0100)
committerLukasz Marek <lukasz.m.luki2@gmail.com>
Sun, 23 Nov 2014 02:33:00 +0000 (03:33 +0100)
Signed-off-by: Lukasz Marek <lukasz.m.luki2@gmail.com>
libavcodec/libvorbisdec.c

index f2c5046..db00572 100644 (file)
@@ -40,7 +40,7 @@ static int oggvorbis_decode_init(AVCodecContext *avccontext) {
 
     if(! avccontext->extradata_size || ! p) {
         av_log(avccontext, AV_LOG_ERROR, "vorbis extradata absent\n");
-        return -1;
+        return AVERROR(EINVAL);
     }
 
     vorbis_info_init(&context->vi) ;
@@ -65,7 +65,7 @@ static int oggvorbis_decode_init(AVCodecContext *avccontext) {
             if(offset >= avccontext->extradata_size - 1) {
                 av_log(avccontext, AV_LOG_ERROR,
                        "vorbis header sizes damaged\n");
-                ret = -1;
+                ret = AVERROR_INVALIDDATA;
                 goto error;
             }
             hsizes[i] += *p;
@@ -84,7 +84,7 @@ static int oggvorbis_decode_init(AVCodecContext *avccontext) {
     } else {
         av_log(avccontext, AV_LOG_ERROR,
                "vorbis initial header len is wrong: %d\n", *p);
-        ret = -1;
+        ret = AVERROR_INVALIDDATA;
         goto error;
     }
 
@@ -94,7 +94,7 @@ static int oggvorbis_decode_init(AVCodecContext *avccontext) {
         context->op.packet = headers[i];
         if(vorbis_synthesis_headerin(&context->vi, &context->vc, &context->op)<0){
             av_log(avccontext, AV_LOG_ERROR, "%d. vorbis header damaged\n", i+1);
-            ret = -1;
+            ret = AVERROR_INVALIDDATA;
             goto error;
         }
     }