[FFmpeg-devel] [PATCH 3/4] lavu/opt: explicity set NULL as dict/binary defaults

Lukasz Marek lukasz.m.luki2 at gmail.com
Sat Nov 8 19:23:42 CET 2014


On 08.11.2014 19:18, Michael Niedermayer wrote:
> On Sat, Nov 08, 2014 at 06:13:52PM +0100, Lukasz Marek wrote:
>> Unless user clears context manually during creation,
>> binary and dict pointer may contain random values.
>
> wouldnt the string type in that case free random pointers ?

Strings are set to their defaults.

> and wouldnt it be inconsistent not to free dict/binary ?

Yes, it is inconsistent, so patch dropped.

BTW, As for as I know binary option may be set from command line, but 
cannot have default. I will add it unless forget.




More information about the ffmpeg-devel mailing list