From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Mon, 27 Aug 2007 01:04:09 +0000 (+0000)
Subject: windows-recursion-fix-20070826
X-Git-Tag: BP-openafs-windows-kdfs-ifs~526
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=cd0a515e9f68604ab0fef15a9b7110402d2bc8a9;p=packages%2Fo%2Fopenafs.git

windows-recursion-fix-20070826


protect against a null pointer dereference
---

diff --git a/src/WINNT/afsd/cm_vnodeops.c b/src/WINNT/afsd/cm_vnodeops.c
index 74e3e4561..efbf2b3c6 100644
--- a/src/WINNT/afsd/cm_vnodeops.c
+++ b/src/WINNT/afsd/cm_vnodeops.c
@@ -1920,7 +1920,9 @@ long cm_NameI(cm_scache_t *rootSCachep, char *pathp, long flags,
                             fids[fid_count++] = nscp->fid;
                         }
                     }
-                } else {
+                }
+
+                if (code) {
 		    cm_ReleaseSCache(tscp);
 		    if (dirScp)
 			cm_ReleaseSCache(dirScp);
@@ -1933,7 +1935,8 @@ long cm_NameI(cm_scache_t *rootSCachep, char *pathp, long flags,
 			osi_Log1(afsd_logp,"cm_NameI code 0x%x", code);
 			return code;
 		    }
-		}	
+		}
+
 		haveComponent = 0;	/* component done */
 		if (dirScp)
 		    cm_ReleaseSCache(dirScp);