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 4361888931deecebfe4263ae8dd6b14dec6afe37..c736706a9eb18fd50a7e8ee5ef1bed8b79be325d 100644 (file)
--- 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;
 }