[MPlayer-cvslog] r30957 - trunk/loader/win32.c

sesse subversion at mplayerhq.hu
Thu Mar 25 13:58:41 CET 2010

Author: sesse
Date: Thu Mar 25 13:58:41 2010
New Revision: 30957

Don't try to delete the global memory mutex in the Win32 loader code,
since it's now statically allocated and will not be reallocated if a new
allocation comes along.

This also fixes an issue where the mutex would not always be properly
unlocked, leading to deadlocks. I thought I'd committed that ages ago,
but obviously not, and it broke CineForm initialization.


Modified: trunk/loader/win32.c
--- trunk/loader/win32.c	Wed Mar 24 21:43:41 2010	(r30956)
+++ trunk/loader/win32.c	Thu Mar 25 13:58:41 2010	(r30957)
@@ -436,10 +436,7 @@ static int my_release(void* memory)
-    if (last_alloc)
-	pthread_mutex_unlock(&memmut);
-    else
-	pthread_mutex_destroy(&memmut);
+    pthread_mutex_unlock(&memmut);
     //if (alccnt < 40000) printf("MY_RELEASE: %p\t%ld    (%d)\n", header, header->size, alccnt);

More information about the MPlayer-cvslog mailing list