Merge pull request #96 from purva-vasudeo/master

Clean old handle before creating a new one
Azure · Mar 4, 2021 · 7e4773a · 7e4773a
2 parents 4fd52bd + 247d1e4
commit 7e4773a
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/iisnode/cprotocolbridge.cpp b/src/iisnode/cprotocolbridge.cpp
@@ -626,6 +626,12 @@ void WINAPI CProtocolBridge::CreateNamedPipeConnection(DWORD error, DWORD bytesT
 
     if (INVALID_HANDLE_VALUE == pipe)
     {
+        // Clean up old pipe handle, otherwise there will be a handle leak
+        HANDLE oldPipe = ctx->GetPipe();
+        if (oldPipe != NULL && oldPipe != INVALID_HANDLE_VALUE) {
+            CloseHandle(oldPipe);
+        }
+
         ErrorIf(INVALID_HANDLE_VALUE == (pipe = CreateFile(
             ctx->GetNodeProcess()->GetNamedPipeName(),
             GENERIC_READ | GENERIC_WRITE,