From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Sun, 4 Jun 2006 07:14:30 +0000 (+0000)
Subject: DEVEL15-viced-swap-old-cbcon-diff-20060603
X-Git-Tag: openafs-devel-1_5_2~10
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=e425a3995cd8e225c354cf28477ab9e9f0321574;p=packages%2Fo%2Fopenafs.git

DEVEL15-viced-swap-old-cbcon-diff-20060603

FIXES 32985

swap cbconn when other side changes IPs


(cherry picked from commit b30dcf9738d7fa87f8da4afa1f024376130b80a9)
---

diff --git a/src/viced/host.c b/src/viced/host.c
index 1ccb4c757..91c4bab84 100644
--- a/src/viced/host.c
+++ b/src/viced/host.c
@@ -1384,6 +1384,8 @@ h_GetHost_r(struct rx_connection *tcon)
 		     * the list of interfaces for the existing host and
 		     * delete the host structure we just allocated. */
 		    if (oldHost->host != haddr || oldHost->port != hport) {
+			struct rx_connection *rxconn;
+
 			ViceLog(25,
 				("CB: new addr %s:%d for old host %s:%d\n",
 				  afs_inet_ntoa_r(haddr, hoststr),
@@ -1419,6 +1421,30 @@ h_GetHost_r(struct rx_connection *tcon)
 			addInterfaceAddr_r(oldHost, haddr, hport);
 			oldHost->host = haddr;
 			oldHost->port = hport;
+			rxconn = oldHost->callback_rxcon;
+			oldHost->callback_rxcon = host->callback_rxcon;
+			host->callback_rxcon = NULL;
+			
+			if (rxconn) {
+			    struct client *client;
+			    /*
+			     * If rx_DestroyConnection calls h_FreeConnection we will
+			     * deadlock on the host_glock_mutex. Work around the problem
+			     * by unhooking the client from the connection before
+			     * destroying the connection.
+			     */
+			    client = rx_GetSpecific(rxconn, rxcon_client_key);
+			    if (client) {
+				if (client->tcon != rxconn) 
+				    ViceLog(0,("CB: client %x tcon %x didn't match rxconn %x\n", client, client->tcon, rxconn));
+				else {
+				    rx_PutConnection(client->tcon);
+				    client->tcon = NULL;
+				}
+			    }
+			    rx_SetSpecific(rxconn, rxcon_client_key, (void *)0);
+			    rx_DestroyConnection(rxconn);
+			}
 		    }
 		    host->hostFlags |= HOSTDELETED;
 		    h_Unlock_r(host);