From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Tue, 10 Oct 2006 16:55:49 +0000 (+0000)
Subject: windows-scache-ref-leak-20061010
X-Git-Tag: BP-openafs-windows-kdfs-ifs~996
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=c60736030bc7f7a678fc44af90d001cd8c08acf5;p=packages%2Fo%2Fopenafs.git

windows-scache-ref-leak-20061010


if cm_FindSCache is called the returned scache object
must be released.
---

diff --git a/src/WINNT/afsd/cm_buf.c b/src/WINNT/afsd/cm_buf.c
index a8a63470e..a5da85778 100644
--- a/src/WINNT/afsd/cm_buf.c
+++ b/src/WINNT/afsd/cm_buf.c
@@ -428,6 +428,8 @@ long buf_SetNBuffers(afs_uint64 nbuffers)
  */
 void buf_WaitIO(cm_scache_t * scp, cm_buf_t *bp)
 {
+    int release = 0;
+
     if (scp)
         osi_assert(scp->magic == CM_SCACHE_MAGIC);
     osi_assert(bp->magic == CM_BUF_MAGIC);
@@ -464,7 +466,8 @@ void buf_WaitIO(cm_scache_t * scp, cm_buf_t *bp)
         }
 
         if ( !scp ) {
-            scp = cm_FindSCache(&bp->fid);
+            if (scp = cm_FindSCache(&bp->fid))
+		 release = 1;
         }
         if ( scp ) {
             lock_ObtainMutex(&scp->mx);
@@ -484,6 +487,9 @@ void buf_WaitIO(cm_scache_t * scp, cm_buf_t *bp)
         osi_Wakeup((LONG_PTR) bp);
     }
     osi_Log1(buf_logp, "WaitIO finished wait for bp 0x%p", bp);
+
+    if (scp && release)
+	cm_ReleaseSCache(scp);
 }
 
 /* find a buffer, if any, for a particular file ID and offset.  Assumes