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

James Zern jzern
Tue Feb 8 00:43:18 CET 2011


On Sat, Feb 5, 2011 at 14:20, Ronald S. Bultje <rsbultje at gmail.com> wrote:
> 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.
>
Yes I thought I'd check on the opinion of that before making that
change as the remainder of the set is still exposed (sws, avformat,
etc.), though those are used.



More information about the ffmpeg-devel mailing list