From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Thu, 4 Aug 2011 21:21:59 +0000 (-0400)
Subject: Windows: fix condition calls to osi_Log
X-Git-Tag: upstream/1.6.1.pre4^2~47
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=36120d812be2a86a094135a9c37a460ec67d7035;p=packages%2Fo%2Fopenafs.git

Windows: fix condition calls to osi_Log

The osi_Log macro is if(foo) osi_AddLog()

If osi_Log macros will be conditionally called, the conditonal
needs to have bracing.

Reviewed-on: http://gerrit.openafs.org/5160
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
(cherry picked from commit 4e42d6fd18097d0c8d2e4b455d3c540743d7dbda)

Change-Id: Ic8063144a5069736c95a57965a28d6a101749b3e
Reviewed-on: http://gerrit.openafs.org/6794
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Derrick Brashear <shadow@dementix.org>
---

diff --git a/src/WINNT/afsd/cm_daemon.c b/src/WINNT/afsd/cm_daemon.c
index 80b8fc312..a99b78f2f 100644
--- a/src/WINNT/afsd/cm_daemon.c
+++ b/src/WINNT/afsd/cm_daemon.c
@@ -170,11 +170,12 @@ void cm_BkgDaemon(void * parm)
             } /* otherwise fall through */
         case 0:  /* success */
         default: /* other error */
-            if (code == 0)
+            if (code == 0) {
                 osi_Log1(afsd_logp,"cm_BkgDaemon SUCCESS: request 0x%p", rp);
-            else
+            } else {
                 osi_Log2(afsd_logp,"cm_BkgDaemon FAILED: request dropped 0x%p code 0x%x",
                          rp, code);
+            }
             cm_ReleaseUser(rp->userp);
             cm_ReleaseSCache(rp->scp);
             free(rp);