On Sun, 20 Sep 2015 21:55:34 -0500
Rodger Combs <[email protected]> wrote:
> ---
> libavformat/hls.c | 12 ++++++++----
> 1 file changed, 8 insertions(+), 4 deletions(-)
>
> diff --git a/libavformat/hls.c b/libavformat/hls.c
> index c16c770..a5073ff 100644
> --- a/libavformat/hls.c
> +++ b/libavformat/hls.c
> @@ -1317,10 +1317,14 @@ static int save_avio_options(AVFormatContext *s)
>
> while (*opt) {
> if (av_opt_get(s->pb, *opt, AV_OPT_SEARCH_CHILDREN, &buf) >= 0) {
> - ret = av_dict_set(&c->avio_opts, *opt, buf,
> - AV_DICT_DONT_STRDUP_VAL);
> - if (ret < 0)
> - return ret;
> + if (strlen(buf)) {
> + ret = av_dict_set(&c->avio_opts, *opt, buf,
> + AV_DICT_DONT_STRDUP_VAL);
> + if (ret < 0)
> + return ret;
> + } else {
> + av_free(buf);
> + }
> }
> opt++;
> }
I guess this looks good, but I don't understand why it's needed.
_______________________________________________
ffmpeg-devel mailing list
[email protected]
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
