]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s4:rpc_server/netlogon: make use of creds->ex->client_sid
authorStefan Metzmacher <metze@samba.org>
Wed, 2 Oct 2024 17:01:39 +0000 (19:01 +0200)
committerDouglas Bagnall <dbagnall@samba.org>
Wed, 30 Oct 2024 23:08:36 +0000 (23:08 +0000)
creds->sid will be removed soon...

BUG: https://bugzilla.samba.org/show_bug.cgi?id=15425

Signed-off-by: Stefan Metzmacher <metze@samba.org>
Reviewed-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
source4/rpc_server/netlogon/dcerpc_netlogon.c

index ad0eb9ac0760b3a8756eac36e5dd0aeafb8fec5e..5b7ca45ba4269da0b62c727bcdd1b8535788206a 100644 (file)
@@ -952,7 +952,7 @@ static NTSTATUS dcesrv_netr_ServerPasswordSet(struct dcesrv_call_state *dce_call
                                                        r->in.credential, r->out.return_authenticator,
                                                        &creds);
        NT_STATUS_NOT_OK_RETURN(nt_status);
-       client_sid = creds->sid;
+       client_sid = &creds->ex->client_sid;
 
        sam_ctx = dcesrv_samdb_connect_as_system(mem_ctx, dce_call);
        if (sam_ctx == NULL) {
@@ -998,7 +998,7 @@ static NTSTATUS dcesrv_netr_ServerPasswordSet2(struct dcesrv_call_state *dce_cal
                                                        r->in.credential, r->out.return_authenticator,
                                                        &creds);
        NT_STATUS_NOT_OK_RETURN(nt_status);
-       client_sid = creds->sid;
+       client_sid = &creds->ex->client_sid;
 
        sam_ctx = dcesrv_samdb_connect_as_system(mem_ctx, dce_call);
        if (sam_ctx == NULL) {
@@ -1349,7 +1349,7 @@ static NTSTATUS dcesrv_netr_LogonSamLogon_base_call(struct dcesrv_netr_LogonSamL
                user_info->netlogon_trust_account.account_name
                        = creds->account_name;
                user_info->netlogon_trust_account.sid
-                       = creds->sid;
+                       = &creds->ex->client_sid;
 
                break;
        default:
@@ -2692,7 +2692,7 @@ static NTSTATUS dcesrv_netr_LogonGetDomainInfo(struct dcesrv_call_state *dce_cal
                talloc_free(frame);
        }
        NT_STATUS_NOT_OK_RETURN(status);
-       client_sid = creds->sid;
+       client_sid = &creds->ex->client_sid;
 
        /* We want to avoid connecting as system. */
        sam_ctx = dcesrv_samdb_connect_as_user(mem_ctx, dce_call);
@@ -3088,7 +3088,7 @@ static NTSTATUS dcesrv_netr_NetrLogonSendToSam(struct dcesrv_call_state *dce_cal
                                                        &creds);
 
        NT_STATUS_NOT_OK_RETURN(nt_status);
-       client_sid = creds->sid;
+       client_sid = &creds->ex->client_sid;
 
        switch (creds->secure_channel_type) {
        case SEC_CHAN_BDC:
@@ -4486,7 +4486,7 @@ static NTSTATUS dcesrv_netr_ServerGetTrustInfo(struct dcesrv_call_state *dce_cal
        if (!NT_STATUS_IS_OK(nt_status)) {
                return nt_status;
        }
-       client_sid = creds->sid;
+       client_sid = &creds->ex->client_sid;
 
        /* TODO: check r->in.server_name is our name */
 
@@ -4685,7 +4685,7 @@ static NTSTATUS dcesrv_netr_DsrUpdateReadOnlyServerDnsRecords(struct dcesrv_call
                                                        r->out.return_authenticator,
                                                        &creds);
        NT_STATUS_NOT_OK_RETURN(nt_status);
-       client_sid = creds->sid;
+       client_sid = &creds->ex->client_sid;
 
        if (creds->secure_channel_type != SEC_CHAN_RODC) {
                return NT_STATUS_ACCESS_DENIED;