From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Wed, 7 Oct 2015 22:09:17 +0000 (-0400)
Subject: Windows: InstNetProvider do not leak key handle
X-Git-Tag: upstream/1.8.0_pre1^2~224
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=cda7556b854a07220326a617a56edcf55769567a;p=packages%2Fo%2Fopenafs.git

Windows: InstNetProvider do not leak key handle

If we open a handle, close it.

Change-Id: I1a5b2308a91f3c66791e65f76ca17ae52d34789f
Reviewed-on: http://gerrit.openafs.org/12050
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
Tested-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/WINNT/afsd/afsd_service.c b/src/WINNT/afsd/afsd_service.c
index 436188893..c736706a9 100644
--- a/src/WINNT/afsd/afsd_service.c
+++ b/src/WINNT/afsd/afsd_service.c
@@ -1266,7 +1266,7 @@ static DWORD
 InstNetProvider(const char *svcname, int bInst, const char *before)
 {
     const char *strOrder = NULL;
-    HKEY hkOrder;
+    HKEY hkOrder = NULL;
     LONG rv;
     DWORD dwSize;
     HANDLE hProcHeap;
@@ -1299,6 +1299,8 @@ InstNetProvider(const char *svcname, int bInst, const char *before)
     }
 
   out:
+    if (hkOrder)
+	RegCloseKey(hkOrder);
     free(strOrder);
     return rv;
 }