Commit 888a8e27 authored by Rob Shearman's avatar Rob Shearman Committed by Alexandre Julliard

secur32: Split the NTLM credential and context handles into separate objects.

This prevents races with two threads using the helper object at the same time on two different context handles, eliminates the need to free the credential handle after freeing the context handles and also prevents a crash caused by not clearing session_key in DeleteSecurityContext.
parent b8dd5839
......@@ -111,7 +111,6 @@ SECURITY_STATUS fork_helper(PNegoHelper *new_helper, const char *prog,
{
*new_helper = helper;
helper->major = helper->minor = helper->micro = -1;
helper->password = NULL;
helper->com_buf = NULL;
helper->com_buf_size = 0;
helper->com_buf_offset = 0;
......
......@@ -61,8 +61,6 @@ typedef struct tag_arc4_info {
typedef struct _NegoHelper {
pid_t helper_pid;
HelperMode mode;
SEC_CHAR *password;
int pwlen;
int pipe_in;
int pipe_out;
int major;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment