Quoting Alexandra Hájková (2015-07-29 11:37:51)
> ---
>  libavformat/asfdec.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/libavformat/asfdec.c b/libavformat/asfdec.c
> index 69c9ecb..46545b3 100644
> --- a/libavformat/asfdec.c
> +++ b/libavformat/asfdec.c
> @@ -542,6 +542,7 @@ static void asf_store_aspect_ratio(AVFormatContext *s, 
> uint8_t st_num, uint8_t *
>          else
>              asf->asf_sd[st_num].aspect_ratio.den = value;
>      }
> +    av_freep(&name);

This variable is now freed in three different places. Seems to me it
would be better if it was always freed in the function that allocated
it.

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to