Re: [FFmpeg-devel] [PATCH 2/2] avcodec/scpr3: fix checking ret value of decode_run_i

2019-05-23 Thread Paul B Mahol
On 5/23/19, Marton Balint  wrote:
> Fixes Coverity CID 1441461.
>
> Signed-off-by: Marton Balint 
> ---
>  libavcodec/scpr3.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/scpr3.c b/libavcodec/scpr3.c
> index 5cfad9f4d2..b4d2e21a17 100644
> --- a/libavcodec/scpr3.c
> +++ b/libavcodec/scpr3.c
> @@ -1007,7 +1007,7 @@ static int decompress_i3(AVCodecContext *avctx,
> uint32_t *dst, int linesize)
>  ret = decode_run_i(avctx, ptype, run, , , clr,
> dst, linesize, , ,
> backstep, off, , );
> -if (run < 0)
> +if (ret < 0)
>  return ret;
>  }
>

OK

> --
> 2.16.4
>
> ___
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

[FFmpeg-devel] [PATCH 2/2] avcodec/scpr3: fix checking ret value of decode_run_i

2019-05-23 Thread Marton Balint
Fixes Coverity CID 1441461.

Signed-off-by: Marton Balint 
---
 libavcodec/scpr3.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/scpr3.c b/libavcodec/scpr3.c
index 5cfad9f4d2..b4d2e21a17 100644
--- a/libavcodec/scpr3.c
+++ b/libavcodec/scpr3.c
@@ -1007,7 +1007,7 @@ static int decompress_i3(AVCodecContext *avctx, uint32_t 
*dst, int linesize)
 ret = decode_run_i(avctx, ptype, run, , , clr,
dst, linesize, , ,
backstep, off, , );
-if (run < 0)
+if (ret < 0)
 return ret;
 }
 
-- 
2.16.4

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".