From 03b971ecc3f02f7ac261f31db434176c6237abfb Mon Sep 17 00:00:00 2001
From: Adam Dickmeiss <adam@indexdata.dk>
Date: Fri, 12 Feb 2010 12:55:26 +0100
Subject: [PATCH] Protect struct host

---
 src/connection.c |   26 ++++++++++++++++++++------
 src/database.c   |    3 +++
 src/host.h       |    1 +
 3 files changed, 24 insertions(+), 6 deletions(-)

diff --git a/src/connection.c b/src/connection.c
index 80ba36b..c1f3ff3 100644
--- a/src/connection.c
+++ b/src/connection.c
@@ -94,16 +94,17 @@ static void remove_connection_from_host(struct connection *con)
 {
     struct connection **conp = &con->host->connections;
     assert(con);
+    yaz_mutex_enter(con->host->mutex);
     while (*conp)
     {
         if (*conp == con)
         {
             *conp = (*conp)->next;
-            return;
+            break;
         }
         conp = &(*conp)->next;
     }
-    assert(*conp == 0);
+    yaz_mutex_leave(con->host->mutex);
 }
 
 // Close connection and recycle structure
@@ -137,8 +138,12 @@ static struct connection *connection_create(struct client *cl,
 
     new = xmalloc(sizeof(*new));
     new->host = host;
+
+    yaz_mutex_enter(host->mutex);
     new->next = new->host->connections;
     new->host->connections = new;
+    yaz_mutex_leave(host->mutex);
+
     new->client = cl;
     new->zproxy = 0;
     client_set_connection(cl, new);
@@ -268,37 +273,43 @@ void connection_release(struct connection *co)
 
 void connect_resolver_host(struct host *host, iochan_man_t iochan_man)
 {
-    struct connection *con = host->connections;
+    struct connection *con;
+
+    yaz_mutex_enter(host->mutex);
+    con = host->connections;
     while (con)
     {
         if (con->state == Conn_Resolving)
         {
             if (!host->ipport) /* unresolved */
             {
+                yaz_mutex_leave(host->mutex);
                 connection_destroy(con);
                 /* start all over .. at some point it will be NULL */
                 con = host->connections;
-                continue;
             }
             else if (!con->client)
             {
+                yaz_mutex_leave(host->mutex);
                 connection_destroy(con);
                 /* start all over .. at some point it will be NULL */
                 con = host->connections;
-                continue;
             }
             else
             {
+                yaz_mutex_leave(host->mutex);
                 connection_connect(con, iochan_man);
                 client_start_search(con->client);
+                con = host->connections;
             }
         }
         else
         {
             yaz_log(YLOG_LOG, "connect_resolver_host: state=%d", con->state);
+            con = con->next;
         }
-        con = con->next;
     }
+    yaz_mutex_leave(host->mutex);
 }
 
 static struct host *connection_get_host(struct connection *con)
@@ -430,6 +441,7 @@ int client_prep_connection(struct client *cl,
     {
         // See if someone else has an idle connection
         // We should look at timestamps here to select the longest-idle connection
+        yaz_mutex_enter(host->mutex);
         for (co = host->connections; co; co = co->next)
             if (connection_is_idle(co) &&
                 (!co->client || client_get_session(co->client) != se) &&
@@ -447,6 +459,7 @@ int client_prep_connection(struct client *cl,
             connection_release(co);
             client_set_connection(cl, co);
             co->client = cl;
+            yaz_mutex_leave(host->mutex);
             co->operation_timeout = operation_timeout;
             co->session_timeout = session_timeout;
             /* tells ZOOM to reconnect if necessary. Disabled becuase
@@ -455,6 +468,7 @@ int client_prep_connection(struct client *cl,
         }
         else
         {
+            yaz_mutex_leave(host->mutex);
             co = connection_create(cl, operation_timeout, session_timeout,
                                    iochan_man);
         }
diff --git a/src/database.c b/src/database.c
index 58cbd7e..b121e95 100644
--- a/src/database.c
+++ b/src/database.c
@@ -95,6 +95,7 @@ static struct host *create_host(const char *hostport, iochan_man_t iochan_man)
     host->hostport = xstrdup(hostport);
     host->connections = 0;
     host->ipport = 0;
+    host->mutex = 0;
 
     if (host_getaddrinfo(host, iochan_man))
     {
@@ -102,6 +103,8 @@ static struct host *create_host(const char *hostport, iochan_man_t iochan_man)
         xfree(host);
         return 0;
     }
+    yaz_mutex_create(&host->mutex);
+
     host->next = hosts;
     hosts = host;
     return host;
diff --git a/src/host.h b/src/host.h
index c76702e..5964af5 100644
--- a/src/host.h
+++ b/src/host.h
@@ -26,6 +26,7 @@ struct host {
     char *ipport;
     struct connection *connections; // All connections to this
     struct host *next;
+    YAZ_MUTEX mutex;
 };
 
 
-- 
1.7.10.4