[vlc-devel] [PATCH v3 2/2] access/http: Share cookies between all playlist items
Antti Ajanki
antti.ajanki at iki.fi
Tue Sep 16 20:22:38 CEST 2014
On 15.09.2014 23:40, Rémi Denis-Courmont wrote:
> So either there is a warranty that when playlist_Destroy() is called, no
> thread can be fetching cookies any longer. In that case, the reference counter
> is pointless. Or there is no such warranty and the code is just wrong.
>
> Hopefully, the earlier. Then don't bother with the reference count.
It is indeed the case that nobody can be fetching the cookies anymore
when the cookie jar is destroyed, because the playlist_Deactivate()
called a few lines before cookie jar destruction in playlist_Destroy()
will stop and join inputs (so there are no active http accesses that
could use cookies). I will post a patch without the reference counting.
Antti
More information about the vlc-devel
mailing list