[vlc-devel] [PATCH] rtsp.c deadlock

Sébastien Escudier sebastien-devel at celeos.eu
Mon Dec 1 08:59:27 CET 2008

Quoting Laurent Aimar <fenrir at via.ecp.fr>:

> > This patch unlock lock_media mutex before sending a vod_MediaControl,
> because we
> > won't access p_sys->media[] anymore at this point.
>  It is invalid. The array is not accessed anymore by the thread but p_media
> is.
> The lock protect both the array AND the element inside it.

I looked again, and I see two possible lock :
lock_media is part of vod_sys_t, and declared as :
    /* List of media */
    vlc_mutex_t lock_media;

So I would say this lock is only for the list (p_sys->media[]) .
To lock the media itself there is another lock inside vod_media_t :

Is my patch really invalid ?


More information about the vlc-devel mailing list