[vlc-devel] [PATCH] decoder: fix NULL dereference when video format is updating
thomas at gllm.fr
Mon Apr 27 19:14:44 CEST 2015
On Mon, Apr 27, 2015, at 18:25, Rémi Denis-Courmont wrote:
> Le lundi 27 avril 2015, 18:08:44 Thomas Guillem a écrit :
> > p_owner->p_vout can be set to NULL by decoder_UpdateVideoFormat that can be
> > called from a different thread than the decoder one (see avcodec).
> If the current reads p_owner->p_vout (at line 861 and also in the calling
> function), then p_owner->p_vout must be read-only.
> So this is a bug in avcodec, or a limitation of the decoder core
> depending on
> your point of view. Either way, the patch does not seem "sufficient" to
OK, it's not sufficient because p_vout should be written only by the
DecoderThread (for now). If it's not the case anymore, we should add
locks everytime we read p_vout from this thread. Am I right ?
> Rémi Denis-Courmont
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
More information about the vlc-devel