]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
Windows: fix condition calls to osi_Log
authorJeffrey Altman <jaltman@your-file-system.com>
Thu, 4 Aug 2011 21:21:59 +0000 (17:21 -0400)
committerDerrick Brashear <shadow@dementix.org>
Sun, 26 Feb 2012 06:27:55 +0000 (22:27 -0800)
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>
src/WINNT/afsd/cm_daemon.c

index 80b8fc3120c466d753b0044eb4a7ba863ee74b58..a99b78f2fc77945a806ff23263a5ae79fe4f304b 100644 (file)
@@ -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);