secur32: Don't allocate context handle in wrapper InitializeSecurityContextA/W…

secur32: Don't allocate context handle in wrapper InitializeSecurityContextA/W if it is the same as the handle passed into the function.

secur32: Don't allocate context handle in wrapper InitializeSecurityContextA/W…
697955c9 · Rob Shearman · Alexandre Julliard · 13468346 · 697955c9
Commit 697955c9 authored Dec 29, 2009 by Rob Shearman Committed by Alexandre Julliard Dec 30, 2009
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 2 deletions

wrapper.c dlls/secur32/wrapper.c +4 -2

No files found.
--- a/dlls/secur32/wrapper.c
+++ b/dlls/secur32/wrapper.c
@@ -283,7 +283,8 @@ SECURITY_STATUS WINAPI InitializeSecurityContextA(
                 cred, ctxt, pszTargetName, fContextReq,
                 Reserved1, TargetDataRep, pInput, Reserved2, phNewContext ? &myCtxt : NULL,
                 pOutput, pfContextAttr, ptsExpiry);
-            if ((ret == SEC_E_OK || ret == SEC_I_CONTINUE_NEEDED) && phNewContext)
+            if ((ret == SEC_E_OK || ret == SEC_I_CONTINUE_NEEDED) &&
+                phNewContext && phNewContext != phContext)
            {
                SECURITY_STATUS ret2;
                ret2 = SECUR32_makeSecHandle(phNewContext, package, &myCtxt);
@@ -346,7 +347,8 @@ SECURITY_STATUS WINAPI InitializeSecurityContextW(
                 cred, ctxt, pszTargetName, fContextReq,
                 Reserved1, TargetDataRep, pInput, Reserved2, phNewContext ? &myCtxt : NULL,
                 pOutput, pfContextAttr, ptsExpiry);
-            if ((ret == SEC_E_OK || ret == SEC_I_CONTINUE_NEEDED) && phNewContext)
+            if ((ret == SEC_E_OK || ret == SEC_I_CONTINUE_NEEDED) &&
+                phNewContext && phNewContext != phContext)
            {
                SECURITY_STATUS ret2;
                ret2 = SECUR32_makeSecHandle(phNewContext, package, &myCtxt);