Re: [FFmpeg-devel] [PATCH] avcodec/pngdec: Pass ret from decode_iccp_chunk()

2020-04-20 Thread Michael Niedermayer 
On Mon, Apr 20, 2020 at 03:04:35PM +0200, Anton Khirnov wrote:
> Quoting Michael Niedermayer (2020-04-18 01:48:47)
> > Found while reviewing a patch fixing a similar issue
> > 
> > Signed-off-by: Michael Niedermayer 
> > ---
> 
> Looks ok

will apply

thx

[...]
-- 
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No human being will ever know the Truth, for even if they happen to say it
by chance, they would not even known they had done so. -- Xenophanes


signature.asc
Description: PGP signature
___
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".

Re: [FFmpeg-devel] [PATCH] avcodec/pngdec: Pass ret from decode_iccp_chunk()

2020-04-20 Thread Anton Khirnov 
Quoting Michael Niedermayer (2020-04-18 01:48:47)
> Found while reviewing a patch fixing a similar issue
> 
> Signed-off-by: Michael Niedermayer 
> ---

Looks ok

-- 
Anton Khirnov
___
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] avcodec/pngdec: Pass ret from decode_iccp_chunk()

2020-04-17 Thread Michael Niedermayer 
Found while reviewing a patch fixing a similar issue

Signed-off-by: Michael Niedermayer 
---
 libavcodec/pngdec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c
index 12d4eb0610..67bfc41b31 100644
--- a/libavcodec/pngdec.c
+++ b/libavcodec/pngdec.c
@@ -1302,7 +1302,7 @@ static int decode_frame_common(AVCodecContext *avctx, 
PNGDecContext *s,
 break;
 }
 case MKTAG('i', 'C', 'C', 'P'): {
-if (decode_iccp_chunk(s, length, p) < 0)
+if ((ret = decode_iccp_chunk(s, length, p)) < 0)
 goto fail;
 break;
 }
-- 
2.17.1

___
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".