[FFmpeg-devel] [PATCH] lavd/lavfi: fix two memleaks.

Michael Niedermayer michaelni at gmx.at
Fri Dec 16 21:50:53 CET 2011


On Thu, Dec 15, 2011 at 07:51:38PM +0100, Nicolas George wrote:
>
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  libavdevice/lavfi.c |    2 ++
>  1 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git a/libavdevice/lavfi.c b/libavdevice/lavfi.c
> index 04dc3b2..cd73234 100644
> --- a/libavdevice/lavfi.c
> +++ b/libavdevice/lavfi.c
> @@ -71,6 +71,7 @@ av_cold static int lavfi_read_close(AVFormatContext *avctx)
>
>      av_freep(&lavfi->sink_stream_map);
>      av_freep(&lavfi->stream_sink_map);
> +    av_freep(&lavfi->sinks);
>      avfilter_graph_free(&lavfi->graph);
>
>      return 0;

ok

> @@ -252,6 +253,7 @@ av_cold static int lavfi_read_header(AVFormatContext *avctx,
>      }
>
>  end:
> +    av_free(pix_fmts);

this might lead to use after free in sink_buffer.c if iam not misssing
anything

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

It is what and why we do it that matters, not just one of them.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111216/7cea658a/attachment.asc>


More information about the ffmpeg-devel mailing list