avcodec/scpr: make sure count and min are valid
authorPaul B Mahol <onemda@gmail.com>
Fri, 7 Sep 2018 13:09:40 +0000 (15:09 +0200)
committerPaul B Mahol <onemda@gmail.com>
Fri, 7 Sep 2018 13:09:40 +0000 (15:09 +0200)
libavcodec/scpr.c

index ee6bf21..f158435 100644 (file)
@@ -529,7 +529,7 @@ static int decompress_p(AVCodecContext *avctx,
         return ret;
 
     max += temp << 8;
-    if (min > max)
+    if (min > max || min >= s->nbcount)
         return AVERROR_INVALIDDATA;
 
     memset(s->blocks, 0, sizeof(*s->blocks) * s->nbcount);
@@ -541,6 +541,8 @@ static int decompress_p(AVCodecContext *avctx,
         ret |= decode_value(s, s->count_model, 256, 20, &count);
         if (ret < 0)
             return ret;
+        if (count <= 0)
+            return AVERROR_INVALIDDATA;
 
         while (min < s->nbcount && count-- > 0) {
             s->blocks[min++] = fill;