From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Mon, 6 Oct 2014 23:20:27 +0000 (-0400)
Subject: Windows: Fix cm_AppendServerList
X-Git-Tag: upstream/1.8.0_pre1^2~472
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=ba0b0986ecd6407fc07d5733ca741c1e5fa7bf52;p=packages%2Fo%2Fopenafs.git

Windows: Fix cm_AppendServerList

Should use || and not | as the operator when testing for
NULL pointers.

Change-Id: I00afe64aec4f965d6a831028b546ed01d8e9672a
Reviewed-on: http://gerrit.openafs.org/11523
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
---

diff --git a/src/WINNT/afsd/cm_server.c b/src/WINNT/afsd/cm_server.c
index 44b928ad3..318dd4c7f 100644
--- a/src/WINNT/afsd/cm_server.c
+++ b/src/WINNT/afsd/cm_server.c
@@ -1532,7 +1532,7 @@ void cm_AppendServerList(cm_serverRef_t *dest, cm_serverRef_t **src)
 {
     cm_serverRef_t *ref;
 
-    if (dest == NULL | src == NULL || *src == NULL)
+    if (dest == NULL || src == NULL || *src == NULL)
 	return;
 
     for (ref = dest; ref->next != NULL; ref = ref->next);