[SCM] Samba Shared Repository - branch master updated

Jeremy Allison jra at samba.org
Wed Jun 20 19:06:03 UTC 2018


The branch, master has been updated
       via  bca97cc Possible memory leak in map_info3_to_validation
      from  defc1ce heimdal: remove include/includedir directives for krb5.conf

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


- Log -----------------------------------------------------------------
commit bca97cc4738fb4461fd27b6df9556f3e1377519d
Author: Swen Schillig <swen at vnet.ibm.com>
Date:   Fri May 25 10:40:54 2018 +0200

    Possible memory leak in map_info3_to_validation
    
    In case of a failing call to copy_netr_SamInfo3, the allocated memory
    for "validation" needs to be free'd before returning.
    
    Signed-off-by: Swen Schillig <swen at vnet.ibm.com>
    Reviewed-by: Jeremy Allison <jra at samba.org>
    Reviewed-by: Christof Schmitt <cs at samba.org>
    
    Autobuild-User(master): Jeremy Allison <jra at samba.org>
    Autobuild-Date(master): Wed Jun 20 21:05:40 CEST 2018 on sn-devel-144

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

Summary of changes:
 source3/rpc_client/util_netlogon.c | 1 +
 1 file changed, 1 insertion(+)


Changeset truncated at 500 lines:

diff --git a/source3/rpc_client/util_netlogon.c b/source3/rpc_client/util_netlogon.c
index c57979a..c5443ba 100644
--- a/source3/rpc_client/util_netlogon.c
+++ b/source3/rpc_client/util_netlogon.c
@@ -353,6 +353,7 @@ NTSTATUS map_info3_to_validation(TALLOC_CTX *mem_ctx,
 				    info3,
 				    &validation->sam3);
 	if (!NT_STATUS_IS_OK(status)) {
+		TALLOC_FREE(validation);
 		return status;
 	}
 


-- 
Samba Shared Repository



More information about the samba-cvs mailing list