The branch, master has been updated via 1ccb69a s3:winbindd: fix query_user for users with NULL full name. via 53d19fa s3:winbindd: fix query_user_list for users with NULL full_name from 05d7524 s4:setup/spn_update_list - the DNS SPN is only used in DNS mode
http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master - Log ----------------------------------------------------------------- commit 1ccb69aabb9057d272fc584d648a4ed9ffc0f3b7 Author: Michael Adam <ob...@samba.org> Date: Mon Nov 1 12:33:43 2010 +0100 s3:winbindd: fix query_user for users with NULL full name. Not sure whether query_user ever sends a NULL full name. Query Display Info does, and it does not harm to add the check here, too. Autobuild-User: Michael Adam <ob...@samba.org> Autobuild-Date: Mon Nov 1 13:03:33 UTC 2010 on sn-devel-104 commit 53d19fac5f97ca234b75046602455653e2a0b2e7 Author: Michael Adam <ob...@samba.org> Date: Mon Nov 1 12:31:34 2010 +0100 s3:winbindd: fix query_user_list for users with NULL full_name ----------------------------------------------------------------------- Summary of changes: source3/winbindd/winbindd_rpc.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) Changeset truncated at 500 lines: diff --git a/source3/winbindd/winbindd_rpc.c b/source3/winbindd/winbindd_rpc.c index 88b1996..47d8f34 100644 --- a/source3/winbindd/winbindd_rpc.c +++ b/source3/winbindd/winbindd_rpc.c @@ -109,7 +109,9 @@ NTSTATUS rpc_query_user_list(TALLOC_CTX *mem_ctx, } dst->full_name = talloc_strdup(info, src->full_name.string); - if (dst->full_name == NULL) { + if ((src->full_name.string != NULL) && + (dst->full_name == NULL)) + { return NT_STATUS_NO_MEMORY; } @@ -500,7 +502,9 @@ NTSTATUS rpc_query_user(TALLOC_CTX *mem_ctx, user_info->full_name = talloc_strdup(user_info, info->info21.full_name.string); - if (user_info->acct_name == NULL) { + if ((info->info21.full_name.string != NULL) && + (user_info->acct_name == NULL)) + { return NT_STATUS_NO_MEMORY; } -- Samba Shared Repository