[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-5285-g47333fc

Karolin Seeger kseeger at samba.org
Sat Jun 6 13:43:54 GMT 2009


The branch, v3-3-test has been updated
       via  47333fc8785457239a499a298536664f152b681d (commit)
      from  11ed212591d612632fcb47f1eac10507b89ffdec (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-3-test


- Log -----------------------------------------------------------------
commit 47333fc8785457239a499a298536664f152b681d
Author: Volker Lendecke <vl at samba.org>
Date:   Thu Mar 12 17:23:17 2009 +0100

    Fix bug 6157
    
    This patch picks the alphabetically smallest one of the multi-value attribute
    "uid". This fixes a regression against 3.0 and also becomes deterministic.

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

Summary of changes:
 source/include/smbldap.h |    3 ++
 source/lib/smbldap.c     |   56 ++++++++++++++++++++++++++++++++++++++++++++++
 source/passdb/pdb_ldap.c |    2 +-
 3 files changed, 60 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/include/smbldap.h b/source/include/smbldap.h
index a2cb8c5..e312bb2 100644
--- a/source/include/smbldap.h
+++ b/source/include/smbldap.h
@@ -212,6 +212,9 @@ const char** get_userattr_list( TALLOC_CTX *mem_ctx, int schema_ver );
 char * smbldap_talloc_single_attribute(LDAP *ldap_struct, LDAPMessage *entry,
 				       const char *attribute,
 				       TALLOC_CTX *mem_ctx);
+char * smbldap_talloc_smallest_attribute(LDAP *ldap_struct, LDAPMessage *entry,
+					 const char *attribute,
+					 TALLOC_CTX *mem_ctx);
 void talloc_autofree_ldapmsg(TALLOC_CTX *mem_ctx, LDAPMessage *result);
 void talloc_autofree_ldapmod(TALLOC_CTX *mem_ctx, LDAPMod **mod);
 const char *smbldap_talloc_dn(TALLOC_CTX *mem_ctx, LDAP *ld,
diff --git a/source/lib/smbldap.c b/source/lib/smbldap.c
index 7e9f1ac..03c6573 100644
--- a/source/lib/smbldap.c
+++ b/source/lib/smbldap.c
@@ -333,6 +333,62 @@ ATTRIB_MAP_ENTRY sidmap_attr_list[] = {
 	return result;
 }
 
+ char * smbldap_talloc_smallest_attribute(LDAP *ldap_struct, LDAPMessage *entry,
+					  const char *attribute,
+					  TALLOC_CTX *mem_ctx)
+{
+	char **values;
+	char *result;
+	size_t converted_size;
+	int i, num_values;
+
+	if (attribute == NULL) {
+		return NULL;
+	}
+
+	values = ldap_get_values(ldap_struct, entry, attribute);
+
+	if (values == NULL) {
+		DEBUG(10, ("attribute %s does not exist\n", attribute));
+		return NULL;
+	}
+
+	if (!pull_utf8_talloc(mem_ctx, &result, values[0], &converted_size)) {
+		DEBUG(10, ("pull_utf8_talloc failed\n"));
+		ldap_value_free(values);
+		return NULL;
+	}
+
+	num_values = ldap_count_values(values);
+
+	for (i=1; i<num_values; i++) {
+		char *tmp;
+
+		if (!pull_utf8_talloc(mem_ctx, &tmp, values[i],
+				      &converted_size)) {
+			DEBUG(10, ("pull_utf8_talloc failed\n"));
+			TALLOC_FREE(result);
+			ldap_value_free(values);
+			return NULL;
+		}
+
+		if (StrCaseCmp(tmp, result) < 0) {
+			TALLOC_FREE(result);
+			result = tmp;
+		} else {
+			TALLOC_FREE(tmp);
+		}
+	}
+
+	ldap_value_free(values);
+
+#ifdef DEBUG_PASSWORDS
+	DEBUG (100, ("smbldap_get_single_attribute: [%s] = [%s]\n",
+		     attribute, result));
+#endif
+	return result;
+}
+
  static int ldapmsg_destructor(LDAPMessage **result) {
 	ldap_msgfree(*result);
 	return 0;
diff --git a/source/passdb/pdb_ldap.c b/source/passdb/pdb_ldap.c
index 554eb6e..bc485e3 100644
--- a/source/passdb/pdb_ldap.c
+++ b/source/passdb/pdb_ldap.c
@@ -559,7 +559,7 @@ static bool init_sam_from_ldap(struct ldapsam_privates *ldap_state,
 		goto fn_exit;
 	}
 
-	if (!(username = smbldap_talloc_single_attribute(priv2ld(ldap_state),
+	if (!(username = smbldap_talloc_smallest_attribute(priv2ld(ldap_state),
 					entry,
 					"uid",
 					ctx))) {


-- 
Samba Shared Repository


More information about the samba-cvs mailing list