You're right, I should at least check if ret == 0. I'll update the patch and check if it works.
Em ter, 22 de set de 2015 às 08:09, Lucas Andrade <lu...@mindello.com.br> escreveu: > I think the other code paths already sets the cookie. > > Em Ter, 22 de set de 2015 04:41, wm4 <nfx...@googlemail.com> escreveu: > >> On Tue, 22 Sep 2015 04:37:48 +0000 >> Lucas Andrade <lu...@mindello.com.br> wrote: >> >> > This fixes the https://trac.ffmpeg.org/ticket/4808, as it sets the >> cookie >> > on playlist response with setcookie header. >> > >> > Update Cookies on Setcookie playlist response >> > --- >> > libavformat/hls.c | 2 ++ >> > 1 file changed, 2 insertions(+) >> > >> > diff --git a/libavformat/hls.c b/libavformat/hls.c >> > index 82dd744..e5c84e1 100644 >> > --- a/libavformat/hls.c >> > +++ b/libavformat/hls.c >> > @@ -996,6 +996,8 @@ static int open_input(HLSContext *c, struct playlist >> > *pls) >> > >> > if (seg->key_type == KEY_NONE) { >> > ret = open_url(pls->parent->priv_data, &pls->input, seg->url, >> > opts); >> > + update_options(&c->cookies, "cookies", pls->input->priv_data); >> > + av_dict_set(&opts, "cookies", c->cookies, 0); >> > } else if (seg->key_type == KEY_AES_128) { >> > // HLSContext *c = var->parent->priv_data; >> > char iv[33], key[33], url[MAX_URL_SIZE]; >> >> This doesn't do the same for the other code paths. >> _______________________________________________ >> ffmpeg-devel mailing list >> ffmpeg-devel@ffmpeg.org >> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel >> > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
