From: Volker Lendecke <vl@samba.org>
Date: Fri, 1 Oct 2021 14:28:57 +0000 (+0200)
Subject: winbind: Don't transfer a pointer that's NULL anyway
X-Git-Tag: tdb-1.4.6~209
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=0eed31db6c327eaedc984ff5aab32afeea8a144a;p=thirdparty%2Fsamba.git

winbind: Don't transfer a pointer that's NULL anyway

ncacn_conn was created by make_internal_ncacn_conn with talloc_zero(),
and that does not set session_info for the purely one-shot connection
state in winbindd_dual_ndrcmd().

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
---

diff --git a/source3/winbindd/winbindd_dual_ndr.c b/source3/winbindd/winbindd_dual_ndr.c
index 36f01224e24..be58b27e79d 100644
--- a/source3/winbindd/winbindd_dual_ndr.c
+++ b/source3/winbindd/winbindd_dual_ndr.c
@@ -432,7 +432,6 @@ static NTSTATUS make_internal_dcesrv_connection(TALLOC_CTX *mem_ctx,
 		status = NT_STATUS_NO_MEMORY;
 		goto fail;
 	}
-	conn->default_auth_state->session_info = ncacn_conn->session_info;
 	conn->default_auth_state->auth_finished = true;
 
 	context = talloc_zero(conn, struct dcesrv_connection_context);