[FFmpeg-devel] [PATCH 1/2] lavf/http.c: Free allocated client URLContext in case of error.

Stephan Holljes klaxa1337 at googlemail.com
Thu Apr 19 23:08:25 EEST 2018


On Fri, Jan 12, 2018 at 7:16 PM, Stephan Holljes
<klaxa1337 at googlemail.com> wrote:
> Signed-off-by: Stephan Holljes <klaxa1337 at googlemail.com>
> ---
>  libavformat/http.c | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/libavformat/http.c b/libavformat/http.c
> index 4806b1e59b..537d0a4773 100644
> --- a/libavformat/http.c
> +++ b/libavformat/http.c
> @@ -574,7 +574,11 @@ static int http_accept(URLContext *s, URLContext **c)
>          goto fail;
>      cc->hd = cl;
>      cc->is_multi_client = 1;
> +    return 0;
>  fail:
> +    if (c) {
> +        ffurl_closep(c);
> +    }
>      return ret;
>  }
>
> --
> 2.15.1
>

Maybe it was not apparent (I'm not too organized with email), but this
patch was sent along with the tcp patch . It was also discussed in
this thread: https://ffmpeg.org/pipermail/ffmpeg-devel/2017-April/210024.html

Thanks!


More information about the ffmpeg-devel mailing list