freemyipod r168 - Code Review

Repository:	freemyipod
Revision:	< r167‎ \| r168 \| r169 >
Date:	22:17, 21 August 2010
Author:	theseven
Status:	new
Tags:
Comment:	Fix a bunch of mutex deadlocks in the button code
Modified paths:	/embios/trunk/button.c (modified) (history)

Diff [purge]

Index: embios/trunk/button.c
—	—	@@ -52,6 +52,7 @@
53	53	button_hooks[i].owner = current_thread;
54	54	button_hooks[i].handler = handler;
55	55	return 0;
	56	+ mutex_unlock(&button_mutex);
56	57	}
57	58	mutex_unlock(&button_mutex);
58	59	return -1;
—	—	@@ -66,6 +67,7 @@
67	68	{
68	69	button_hooks[i].owner = NULL;
69	70	button_hooks[i].handler = NULL;
	71	+ mutex_unlock(&button_mutex);
70	72	return 0;
71	73	}
72	74	mutex_unlock(&button_mutex);
—	—	@@ -89,6 +91,7 @@
90	92	{
91	93	button_hooks[i].owner = NULL;
92	94	button_hooks[i].handler = NULL;
	95	+ mutex_unlock(&button_mutex);
93	96	}
94	97	mutex_unlock(&button_mutex);
95	98	}