[FFmpeg-devel] [PATCH 1/2] cmdutils: fix opt_values leak

Ronald S. Bultje rsbultje
Sat Feb 5 23:20:45 CET 2011


Hi,

On Fri, Feb 4, 2011 at 8:58 PM, James Zern <jzern at google.com> wrote:
> Add free to uninit_opts and relocate opt_names to same
> ---
> ?cmdutils.c | ? ?2 ++
> ?ffmpeg.c ? | ? ?1 -
> ?2 files changed, 2 insertions(+), 1 deletions(-)
>
> In addition to this it is possible that opt_names could be removed
> from cmdutils.h as it's unused elsewhere.

Looks OK to me, queued locally. You could also consider making
opt_names[] and opt_values[] (and possibly others) static if you
remove them from cmdutils.h.

Ronald



More information about the ffmpeg-devel mailing list