[FFmpeg-devel] [PATCH 3/4] vsrc_testsrc: free option values.

Stefano Sabatini stefasab at gmail.com
Sat Mar 3 14:50:57 CET 2012


On date Saturday 2012-03-03 13:45:41 +0100, Nicolas George encoded:
> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
>  libavfilter/vsrc_testsrc.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/libavfilter/vsrc_testsrc.c b/libavfilter/vsrc_testsrc.c
> index 0839a2b..4ba0b52 100644
> --- a/libavfilter/vsrc_testsrc.c
> +++ b/libavfilter/vsrc_testsrc.c
> @@ -88,17 +88,20 @@ static av_cold int init(AVFilterContext *ctx, const char *args, void *opaque)
>          av_log(ctx, AV_LOG_ERROR, "Invalid frame size: '%s'\n", test->size);
>          return ret;
>      }
> +    av_freep(&test->size);
>  
>      if ((ret = av_parse_video_rate(&frame_rate_q, test->rate)) < 0 ||
>          frame_rate_q.den <= 0 || frame_rate_q.num <= 0) {
>          av_log(ctx, AV_LOG_ERROR, "Invalid frame rate: '%s'\n", test->rate);
>          return ret;
>      }
> +    av_freep(&test->rate);
>  
>      if ((test->duration) && (ret = av_parse_time(&duration, test->duration, 1)) < 0) {
>          av_log(ctx, AV_LOG_ERROR, "Invalid duration: '%s'\n", test->duration);
>          return ret;
>      }
> +    av_freep(&test->duration);
>  
>      if (test->nb_decimals && strcmp(ctx->filter->name, "testsrc")) {
>          av_log(ctx, AV_LOG_WARNING,

LGTM.
-- 
FFmpeg = Fancy and Frenzy Minimal Programmable Empowered Governor


More information about the ffmpeg-devel mailing list