Re: [FFmpeg-devel] [PATCH 16/30] avcodec/indeo5: Fix memleaks upon allocation error

2020-09-15 Thread Paul B Mahol
On Tue, Sep 15, 2020 at 09:39:46AM +0200, Andreas Rheinhardt wrote:
> ff_ivi_init_planes() might error out after having allocated some arrays.
> Set the FF_CODEC_CAP_INIT_CLEANUP flag in order to free these arrays in
> this case.
> 
> Signed-off-by: Andreas Rheinhardt 
> ---
>  libavcodec/indeo5.c | 2 ++
>  1 file changed, 2 insertions(+)

looks ok

> 
> diff --git a/libavcodec/indeo5.c b/libavcodec/indeo5.c
> index 4ccdbcaf0a..ac15d31fef 100644
> --- a/libavcodec/indeo5.c
> +++ b/libavcodec/indeo5.c
> @@ -30,6 +30,7 @@
>  #define BITSTREAM_READER_LE
>  #include "avcodec.h"
>  #include "get_bits.h"
> +#include "internal.h"
>  #include "ivi.h"
>  #include "ivi_dsp.h"
>  #include "indeo5data.h"
> @@ -692,4 +693,5 @@ AVCodec ff_indeo5_decoder = {
>  .close  = ff_ivi_decode_close,
>  .decode = ff_ivi_decode_frame,
>  .capabilities   = AV_CODEC_CAP_DR1,
> +.caps_internal  = FF_CODEC_CAP_INIT_CLEANUP,
>  };
> -- 
> 2.25.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".
___
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 16/30] avcodec/indeo5: Fix memleaks upon allocation error

2020-09-15 Thread Andreas Rheinhardt
ff_ivi_init_planes() might error out after having allocated some arrays.
Set the FF_CODEC_CAP_INIT_CLEANUP flag in order to free these arrays in
this case.

Signed-off-by: Andreas Rheinhardt 
---
 libavcodec/indeo5.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/libavcodec/indeo5.c b/libavcodec/indeo5.c
index 4ccdbcaf0a..ac15d31fef 100644
--- a/libavcodec/indeo5.c
+++ b/libavcodec/indeo5.c
@@ -30,6 +30,7 @@
 #define BITSTREAM_READER_LE
 #include "avcodec.h"
 #include "get_bits.h"
+#include "internal.h"
 #include "ivi.h"
 #include "ivi_dsp.h"
 #include "indeo5data.h"
@@ -692,4 +693,5 @@ AVCodec ff_indeo5_decoder = {
 .close  = ff_ivi_decode_close,
 .decode = ff_ivi_decode_frame,
 .capabilities   = AV_CODEC_CAP_DR1,
+.caps_internal  = FF_CODEC_CAP_INIT_CLEANUP,
 };
-- 
2.25.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".