diff options
author | Joseph Sutton <josephsutton@catalyst.net.nz> | 2022-02-22 14:15:43 +1300 |
---|---|---|
committer | Stefan Metzmacher <metze@samba.org> | 2022-03-01 17:11:35 +0000 |
commit | ef95fb439237910b945b8d6a3ad4a140a8d6d1ea (patch) | |
tree | ad718ec2ac1b41b4e4d744d1d629a114b0b57b2e /auth | |
parent | f6fe86924c2ca756083d3628d5dbace0b12d06b0 (diff) | |
download | samba-ef95fb439237910b945b8d6a3ad4a140a8d6d1ea.tar.gz |
auth: Cope with NULL upn_name in PAC
BUG: https://bugzilla.samba.org/show_bug.cgi?id=14995
Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'auth')
-rw-r--r-- | auth/auth_sam_reply.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/auth/auth_sam_reply.c b/auth/auth_sam_reply.c index b5b6362dc93..fda014c87d5 100644 --- a/auth/auth_sam_reply.c +++ b/auth/auth_sam_reply.c @@ -616,11 +616,13 @@ NTSTATUS make_user_info_dc_pac(TALLOC_CTX *mem_ctx, } if (pac_upn_dns_info != NULL) { - user_info_dc->info->user_principal_name = - talloc_strdup(user_info_dc->info, - pac_upn_dns_info->upn_name); - if (user_info_dc->info->user_principal_name == NULL) { - return NT_STATUS_NO_MEMORY; + if (pac_upn_dns_info->upn_name != NULL) { + user_info_dc->info->user_principal_name = + talloc_strdup(user_info_dc->info, + pac_upn_dns_info->upn_name); + if (user_info_dc->info->user_principal_name == NULL) { + return NT_STATUS_NO_MEMORY; + } } user_info_dc->info->dns_domain_name = |