[vlc-devel] commit: Fix the update system, based on funman patch + some modifications. ( [UTF-8?] RГ©mi Duraffort )

[Kaloyan Kovachev]

On Thu, 26 Jun 2008 00:01:47 +0300, [UTF-8?]RГ©mi Denis-Courmont wrote
> Le mercredi 25 juin 2008 23:42:57 git version control, vous avez
[UTF-8?]Г©critВ :
> > +    vlc_mutex_unlock( &p_update->lock );
> >      vlc_mutex_destroy( &p_update->lock );
> 
> Having these two lines one after the other is almost always a bug.
> 

I did the same in ts_mux :( This patch should fix it and also includes the
previous fix (which is not yet applyed) for setting p_sys->csa to NULL after
delete()

> If someone else is waiting on the mutex, then it will crash.
> If nobody is waiting on the mutex, there is no need to lock it.
> 
> -- 
> [UTF-8?]RГ©mi Denis-Courmont
> http://www.remlab.net/
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Small-fixes-for-ts_mux.patch
Type: text/x-patch
Size: 1301 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20080627/7da684e3/attachment.bin>