[vlc-devel] fix #14586 (6f3e18d5) leads to crash on Android

Andrey Gursky andrey.gursky at e-mail.ua
Sun May 29 20:48:34 CEST 2016


On Sun, 29 May 2016 21:43:38 +0300
Rémi Denis-Courmont <remi at remlab.net> wrote:

> On Sunday 29 May 2016 20:31:37 Andrey Gursky wrote:
> > considering src/android/thread.c:
> > 
> > static __thread struct vlc_thread *thread = NULL;
> > 
> > vlc_thread_t vlc_thread_self (void)
> > {
> >     return thread;
> > }
> > 
> > void vlc_control_cancel(int cmd, ...)
> > {
> >     vlc_thread_t th = vlc_thread_self();
> >     va_list ap;
> > //...
> > }
> > 
> > I don't see where this TLS variable "thread" is assigned before call to
> > vlc_control_cancel().
> 
> TBH, that it´s not hard to find by looking for "thread =", and it has not 
> changed in 3 years.

Yes, I did that, of course. From what I found, it doesn't get assigned.

Regards,
Andrey


More information about the vlc-devel mailing list