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

Ronald S. Bultje rsbultje
Tue Feb 8 02:36:30 CET 2011


Hi,

On Mon, Feb 7, 2011 at 6:43 PM, James Zern <jzern at google.com> wrote:
> On Sat, Feb 5, 2011 at 14:20, Ronald S. Bultje <rsbultje at gmail.com> wrote:
>> 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.

It's fine with me to make it static. Michael, objections?

Ronald



More information about the ffmpeg-devel mailing list