[vlc-devel] [PATCH] Handle GET_PARAMETER keepalive responses during PAUSE
paul at packetship.com
Wed Jun 24 19:42:23 CEST 2015
On 24/06/15 18:30, Rémi Denis-Courmont wrote:
> Le mercredi 24 juin 2015, 18:18:03 Paul Clark a écrit :
>> + if( !wait_Live555_response( p_demux ) )
> Are you sure this function is re-entrant? Seems like nothing good will happen
> if the timeout thread and the input thread end up waiting at the same time.
Since there's a single shared live555 RTSPClient and based on other
/* Voodoo (= no) thread safety here! *Ahem* */
... I'm pretty sure it *isn't* re-entrant, so yes, that would be Bad if
it happened. However this code is only activated during pause and the
existing code attempts to make sure it doesn't conflict with foreground
processing through the b_handle_keep_alive flag.
So the best I can claim is that it is no worse than what is there
already and does fix a nasty lockup... Ensuring provable thread-safety
of this stuff is beyond my understanding of VLC internals I'm afraid.
More information about the vlc-devel