From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Mon, 7 May 2012 04:56:11 +0000 (-0400)
Subject: Windows: Correct refcount leak
X-Git-Tag: openafs-stable-1_8_0pre1~2437
X-Git-Url: https://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=542dad75733354a6fb1e912d3ff75d414459078b

Windows: Correct refcount leak

The new duplicate avoidance code in cm_QueueBKGRequest() leaked
a cm_scache_t and a cm_user_t reference count.

Change-Id: Iceaf32a8d20cf350024fe40e3185c723f6203676
Reviewed-on: http://gerrit.openafs.org/7365
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@secure-endpoints.com>
---

diff --git a/src/WINNT/afsd/cm_daemon.c b/src/WINNT/afsd/cm_daemon.c
index fffbdb8..c93e4bf 100644
--- a/src/WINNT/afsd/cm_daemon.c
+++ b/src/WINNT/afsd/cm_daemon.c
@@ -313,6 +313,8 @@ void cm_QueueBKGRequest(cm_scache_t *scp, cm_bkgProc_t *procp, afs_uint32 p1, af
     lock_ReleaseWrite(&cm_daemonLockp[daemonID]);
 
     if (duplicate) {
+        cm_ReleaseSCache(scp);
+        cm_ReleaseUser(userp);
         free(rp);
     } else {
         osi_Wakeup((LONG_PTR) &cm_bkgListpp[daemonID]);