From ba0b0986ecd6407fc07d5733ca741c1e5fa7bf52 Mon Sep 17 00:00:00 2001 From: Jeffrey Altman Date: Mon, 6 Oct 2014 19:20:27 -0400 Subject: [PATCH] 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 Tested-by: BuildBot --- src/WINNT/afsd/cm_server.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/WINNT/afsd/cm_server.c b/src/WINNT/afsd/cm_server.c index 44b928a..318dd4c 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); -- 1.9.4