From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Mon, 17 Feb 2014 06:12:37 +0000 (-0500)
Subject: Windows: cm_ForceNewConnections serverp == NULL
X-Git-Tag: openafs-stable-1_8_0pre1~780
X-Git-Url: http://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=8803c35994eb48605e26831296bd1c57f0adf50a;hp=4e3ceaccd9dc2b6e6a20e938d82af1ebaa2c43c8

Windows: cm_ForceNewConnections serverp == NULL

If serverp == NULL, return immediately.  Do not dereference a NULL
pointer.

Change-Id: I47781a03f22a83289a23b30ff375249fec18ff51
Reviewed-on: http://gerrit.openafs.org/10845
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Perry Ruiter <pruiter@sinenomine.net>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/WINNT/afsd/cm_conn.c b/src/WINNT/afsd/cm_conn.c
index 8975023..6a867fd 100644
--- a/src/WINNT/afsd/cm_conn.c
+++ b/src/WINNT/afsd/cm_conn.c
@@ -1827,6 +1827,9 @@ void cm_ForceNewConnections(cm_server_t *serverp)
 {
     cm_conn_t *tcp;
 
+    if (serverp == NULL)
+	return;
+
     lock_ObtainWrite(&cm_connLock);
     for (tcp = serverp->connsp; tcp; tcp=tcp->nextp) {
 	lock_ObtainMutex(&tcp->mx);