[FFmpeg-devel] [PATCH 1/3] http: do not print a warning message for expired cookies

wm4 nfxjfg at googlemail.com
Sun Mar 18 14:52:10 EET 2018


On Thu,  8 Mar 2018 04:53:55 +0100
wm4 <nfxjfg at googlemail.com> wrote:

> libavformat prints a warning that the cookie couldn't be parsed (see
> callers of parse_cookie()). This is obviously not true - it could be
> parsed, but was simply ignored. Don't return an error to avoid the
> warning.
> ---
>  libavformat/http.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/http.c b/libavformat/http.c
> index 344fd603cb..d7a72e7129 100644
> --- a/libavformat/http.c
> +++ b/libavformat/http.c
> @@ -802,7 +802,7 @@ static int parse_cookie(HTTPContext *s, const char *p, AVDictionary **cookies)
>              // if the cookie has already expired ignore it
>              if (av_timegm(&new_tm) < av_gettime() / 1000000) {
>                  av_dict_free(&new_params);
> -                return -1;
> +                return 0;
>              }
>  
>              // only replace an older cookie with the same name

All pushed.


More information about the ffmpeg-devel mailing list