[SCM] Samba Shared Repository - branch master updated

Michael Adam obnox at samba.org
Mon Nov 1 07:04:01 MDT 2010


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 <obnox at 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 <obnox at samba.org>
    Autobuild-Date: Mon Nov  1 13:03:33 UTC 2010 on sn-devel-104

commit 53d19fac5f97ca234b75046602455653e2a0b2e7
Author: Michael Adam <obnox at 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


More information about the samba-cvs mailing list