[FFmpeg-devel] [PATCH 08/11] ffserver: export recommented encoder configuration

Lukasz Marek lukasz.m.luki2 at gmail.com
Sat Nov 22 01:12:28 CET 2014


On 18.11.2014 22:50, Reynaldo H. Verdejo Pinochet wrote:
> Hi
>
> On 11/16/2014 10:46 PM, Lukasz Marek wrote:
>> [..]
>> @@ -3355,6 +3354,9 @@ static int add_av_stream(FFServerStream *feed, AVStream *st)
>>       fst = add_av_stream1(feed, av, 0);
>>       if (!fst)
>>           return -1;
>> +    if (av_stream_get_recommended_encoder_configuration(st))
>> +        av_stream_set_recommended_encoder_configuration(fst,
>> +            av_strdup(av_stream_get_recommended_encoder_configuration(st)));
>
> Is the return of av_strdup here been freed somewhere?. Also
> adding braces to ifs when the body is multilined wouldn't
> hurt. Not a blocker of course.
>
> Looks OK otherwise. Feel free to push after confirming ^

Updated patch, av_dict_serialize -> av_dict_get_string



More information about the ffmpeg-devel mailing list