[vlc-devel] [PATCH 5/5] libvlc: player: use vlc_player

Thomas Guillem thomas at gllm.fr
Thu May 9 19:58:50 CEST 2019



On Thu, May 9, 2019, at 19:08, Rémi Denis-Courmont wrote:
> Le torstaina 9. toukokuuta 2019, 19.51.09 EEST Thomas Guillem a écrit :
> > > Application should (still) be allowed to call this, and most* other
> > > functions further down the patch, from within event callbacks. But this
> > > leads to deadlock or abort depending on build type.
> > > 
> > > * Exceptions exist such _stop(), which would necessarily deadlock.
> > 
> > Not anymore, stop is asynchronous.
> 
> There should be a separate asynchronous stop (request) function, notably so 
> that stop can be requested from within callbacks. And in hindsight, we 
> probably should have done it that way from the beginning. But we did not have 
> the benefit of hindsight and we specified stop to wait instead.
> 
> We cannot change it now, because we don't _silently_ break APIs. This would 
> cause huge a number of use-after-free bugs in existing apps.
> 
> If you change the semantics, change the name and/or the number of parameters 
> so that it won't fail silently.

Indeed, we should add libvlc_player_stop_async that call directly vlc_player_stop and have the legacy call condwaiting for the stop event. 

> 
> -- 
> Rémi Denis-Courmont
> http://www.remlab.net/
> 
> 
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list