tiff: Return more meaningful error codes
authorHimangi Saraogi <himangi774@gmail.com>
Sun, 29 Mar 2015 18:57:56 +0000 (00:27 +0530)
committerDiego Biurrun <diego@biurrun.de>
Mon, 30 Mar 2015 15:51:20 +0000 (17:51 +0200)
Signed-off-by: Diego Biurrun <diego@biurrun.de>
libavcodec/tiffenc.c

index 169360f..a020f5f 100644 (file)
@@ -153,7 +153,8 @@ static int add_entry1(TiffEncoderContext *s,
  * @param dst Output buffer
  * @param n Size of input buffer
  * @param compr Compression method
- * @return Number of output bytes. If an output error is encountered, -1 returned
+ * @return Number of output bytes. If an output error is encountered, a negative
+ * value corresponding to an AVERROR error code is returned.
  */
 static int encode_strip(TiffEncoderContext *s, const int8_t *src,
                         uint8_t *dst, int n, int compr)
@@ -166,14 +167,14 @@ static int encode_strip(TiffEncoderContext *s, const int8_t *src,
         unsigned long zlen = s->buf_size - (*s->buf - s->buf_start);
         if (compress(dst, &zlen, src, n) != Z_OK) {
             av_log(s->avctx, AV_LOG_ERROR, "Compressing failed\n");
-            return -1;
+            return AVERROR_UNKNOWN;
         }
         return zlen;
     }
 #endif
     case TIFF_RAW:
         if (check_size(s, n))
-            return -1;
+            return AVERROR(EINVAL);
         memcpy(dst, src, n);
         return n;
     case TIFF_PACKBITS:
@@ -182,7 +183,7 @@ static int encode_strip(TiffEncoderContext *s, const int8_t *src,
     case TIFF_LZW:
         return ff_lzw_encode(s->lzws, src, n);
     default:
-        return -1;
+        return AVERROR(EINVAL);
     }
 }
 
@@ -291,7 +292,7 @@ static int encode_frame(AVCodecContext *avctx, AVPacket *pkt,
     default:
         av_log(s->avctx, AV_LOG_ERROR,
                "This colors format is not supported\n");
-        return -1;
+        return AVERROR(EINVAL);
     }
 
     if (s->compr == TIFF_DEFLATE       ||