[SCM] Samba Shared Repository - branch master updated

Simo Sorce idra at samba.org
Wed Jul 21 17:14:12 MDT 2010


The branch, master has been updated
       via  3b502d8... s3-dcerpc: Fix potential memleak
      from  a3713d7... s3-dcerpc: Properly initialize output buffers on nak/fault

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 3b502d899d8924d285e72f8a9fe666865d6f6926
Author: Simo Sorce <idra at samba.org>
Date:   Sat Jul 17 16:01:07 2010 -0400

    s3-dcerpc: Fix potential memleak
    
    Use a free function for schannel data too for now.

-----------------------------------------------------------------------

Summary of changes:
 source3/rpc_server/srv_pipe.c |    7 ++++++-
 1 files changed, 6 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/rpc_server/srv_pipe.c b/source3/rpc_server/srv_pipe.c
index b25f122..508cb3f 100644
--- a/source3/rpc_server/srv_pipe.c
+++ b/source3/rpc_server/srv_pipe.c
@@ -89,6 +89,11 @@ static void free_pipe_ntlmssp_auth_data(struct pipe_auth_data *auth)
 	TALLOC_FREE(auth->a_u.auth_ntlmssp_state);
 }
 
+static void free_pipe_schannel_auth_data(struct pipe_auth_data *auth)
+{
+	TALLOC_FREE(auth->a_u.schannel_auth);
+}
+
 static DATA_BLOB generic_session_key(void)
 {
 	return data_blob("SystemLibraryDTC", 16);
@@ -1100,7 +1105,7 @@ static bool pipe_schannel_auth_bind(pipes_struct *p,
 		neg.oem_netbios_domain.a, neg.oem_netbios_computer.a));
 
 	/* We're finished with this bind - no more packets. */
-	p->auth.auth_data_free_func = NULL;
+	p->auth.auth_data_free_func = &free_pipe_schannel_auth_data;
 	p->auth.auth_type = PIPE_AUTH_TYPE_SCHANNEL;
 
 	p->pipe_bound = True;


-- 
Samba Shared Repository


More information about the samba-cvs mailing list