# HG changeset patch
# User Adam Chlipala <adam@chlipala.net>
# Date 1326234768 18000
# Node ID ea131de361d97ae7ac986ccb2a416dacf93afb70
# Parent  a54d223c3a7fe8979a5d9f08abe91dd7ebf185a0
Fix some locking issues for client array

diff -r a54d223c3a7f -r ea131de361d9 src/c/cgi.c
--- a/src/c/cgi.c	Tue Jan 10 15:34:18 2012 -0500
+++ b/src/c/cgi.c	Tue Jan 10 17:32:48 2012 -0500
@@ -131,8 +131,7 @@
   if (uw_get_app(ctx)->db_begin(ctx))
     uw_error(ctx, FATAL, "Error running SQL BEGIN");
   uw_get_app(ctx)->expunger(ctx, cli);
-  if (uw_get_app(ctx)->db_commit(ctx))
-    uw_error(ctx, FATAL, "Error running SQL COMMIT");
+  uw_commit(ctx);
 }
 
 void uw_post_expunge(uw_context ctx, void *data) {
diff -r a54d223c3a7f -r ea131de361d9 src/c/fastcgi.c
--- a/src/c/fastcgi.c	Tue Jan 10 15:34:18 2012 -0500
+++ b/src/c/fastcgi.c	Tue Jan 10 17:32:48 2012 -0500
@@ -632,8 +632,7 @@
   if (uw_get_app(ctx)->db_begin(ctx))
     uw_error(ctx, FATAL, "Error running SQL BEGIN");
   uw_get_app(ctx)->expunger(ctx, cli);
-  if (uw_get_app(ctx)->db_commit(ctx))
-    uw_error(ctx, FATAL, "Error running SQL COMMIT");
+  uw_commit(ctx);
 }
 
 void uw_post_expunge(uw_context ctx, void *data) {
diff -r a54d223c3a7f -r ea131de361d9 src/c/http.c
--- a/src/c/http.c	Tue Jan 10 15:34:18 2012 -0500
+++ b/src/c/http.c	Tue Jan 10 17:32:48 2012 -0500
@@ -354,8 +354,7 @@
   if (uw_get_app(ctx)->db_begin(ctx))
     uw_error(ctx, FATAL, "Error running SQL BEGIN");
   uw_get_app(ctx)->expunger(ctx, cli);
-  if (uw_get_app(ctx)->db_commit(ctx))
-    uw_error(ctx, FATAL, "Error running SQL COMMIT");
+  uw_commit(ctx);
 }
 
 void uw_post_expunge(uw_context ctx, void *data) {
diff -r a54d223c3a7f -r ea131de361d9 src/c/urweb.c
--- a/src/c/urweb.c	Tue Jan 10 15:34:18 2012 -0500
+++ b/src/c/urweb.c	Tue Jan 10 17:32:48 2012 -0500
@@ -159,7 +159,7 @@
 static client **clients, *clients_free, *clients_used;
 static unsigned n_clients;
 
-static pthread_mutex_t clients_mutex = PTHREAD_MUTEX_INITIALIZER;
+static pthread_mutex_t clients_mutex = PTHREAD_RECURSIVE_MUTEX_INITIALIZER_NP;
 
 size_t uw_messages_max = SIZE_MAX;
 size_t uw_clients_max = SIZE_MAX;
@@ -177,9 +177,10 @@
     c = clients_free;
     clients_free = clients_free->next;
   }
-  else if (n_clients >= uw_clients_max)
+  else if (n_clients >= uw_clients_max) {
+    pthread_mutex_unlock(&clients_mutex);
     return NULL;
-  else {
+  } else {
     ++n_clients;
     clients = realloc(clients, sizeof(client) * n_clients);
     c = malloc(sizeof(client));