On 12/30/2017 12:42 PM, Steven Liu wrote: > av_opt_get(v->input, "http_version", AV_OPT_SEARCH_CHILDREN, > &http_version_opt); > c->http_multiple = http_version_opt && strncmp((const char > *)http_version_opt, "1.1", 3) == 0; > + av_free(http_version_opt);
Looks OK, but the return value for av_opt_get should also be checked. It can fail with, for example, AVERROR(ENOMEM). - Derek _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
