Ken Thomases : secur32: Clean up the helper in all cases where it was allocated.

Alexandre Julliard julliard at winehq.org
Wed Nov 30 14:19:19 CST 2011


Module: wine
Branch: master
Commit: f4ac641af55be232781018ff3939132c7f86df81
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=f4ac641af55be232781018ff3939132c7f86df81

Author: Ken Thomases <ken at codeweavers.com>
Date:   Tue Nov 29 19:16:13 2011 -0600

secur32: Clean up the helper in all cases where it was allocated.

---

 dlls/secur32/ntlm.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/dlls/secur32/ntlm.c b/dlls/secur32/ntlm.c
index 4c353c3..c1e4466 100644
--- a/dlls/secur32/ntlm.c
+++ b/dlls/secur32/ntlm.c
@@ -694,7 +694,10 @@ static SECURITY_STATUS SEC_ENTRY ntlm_InitializeSecurityContextW(
             lstrcpynA(buffer, want_flags, max_len-1);
             if((ret = run_helper(helper, buffer, max_len, &buffer_len)) 
                     != SEC_E_OK)
+            {
+                cleanup_helper(helper);
                 goto isc_end;
+            }
             if(!strncmp(buffer, "BH", 2))
                 ERR("Helper doesn't understand new command set. Expect more things to fail.\n");
         }




More information about the wine-cvs mailing list