Windows: Fix cm_AppendServerList
authorJeffrey Altman <jaltman@your-file-system.com>
Mon, 6 Oct 2014 23:20:27 +0000 (19:20 -0400)
committerJeffrey Altman <jaltman@your-file-system.com>
Tue, 25 Nov 2014 16:55:01 +0000 (11:55 -0500)
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>

src/WINNT/afsd/cm_server.c

index 44b928a..318dd4c 100644 (file)
@@ -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);