[SCM] Samba Shared Repository - branch master updated

Andrew Tridgell tridge at samba.org
Thu Oct 15 18:57:42 MDT 2009


The branch, master has been updated
       via  7bcc0b2... s4-winsrepl: don't put in attributes with no elements
      from  3f80b06... s3-selftest: enable WINBIND-WBCLIENT against s3.

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 7bcc0b29662663be75828f2618a5013e7c2c2817
Author: Andrew Tridgell <tridge at samba.org>
Date:   Fri Oct 16 11:41:52 2009 +1100

    s4-winsrepl: don't put in attributes with no elements
    
    Empty attributes are no longer allowed by ldb. This also fixes the
    error checking in winsdb_message()
    
    This fixes the samba4.nbt.winsreplication test

-----------------------------------------------------------------------

Summary of changes:
 source4/nbt_server/wins/winsdb.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source4/nbt_server/wins/winsdb.c b/source4/nbt_server/wins/winsdb.c
index 5c3efe2..389995e 100644
--- a/source4/nbt_server/wins/winsdb.c
+++ b/source4/nbt_server/wins/winsdb.c
@@ -794,11 +794,11 @@ static struct ldb_message *winsdb_message(struct ldb_context *ldb,
 	for (i=0;rec->addresses[i];i++) {
 		ret |= ldb_msg_add_winsdb_addr(msg, rec, "address", rec->addresses[i]);
 	}
-	ret |= ldb_msg_add_empty(msg, "registeredBy", 0, NULL);
 	if (rec->registered_by) {
+		ret |= ldb_msg_add_empty(msg, "registeredBy", 0, NULL);
 		ret |= ldb_msg_add_string(msg, "registeredBy", rec->registered_by);
-		if (ret != 0) goto failed;
 	}
+	if (ret != 0) goto failed;
 	return msg;
 
 failed:


-- 
Samba Shared Repository


More information about the samba-cvs mailing list