svn commit: samba r24260 - in branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules: .

[abartlet]

Author: abartlet
Date: 2007-08-07 03:35:45 +0000 (Tue, 07 Aug 2007)
New Revision: 24260

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=24260

Log:
Ensure we always override any existing values for these generated
attributes.  Anything else leads to madness.

Andrew Bartlett

Modified:
   branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules/kludge_acl.c


Changeset:
Modified: branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules/kludge_acl.c
===================================================================
--- branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules/kludge_acl.c       
2007-08-07 03:33:26 UTC (rev 24259)
+++ branches/SAMBA_4_0/source/dsdb/samdb/ldb_modules/kludge_acl.c       
2007-08-07 03:35:45 UTC (rev 24260)
@@ -122,6 +122,8 @@
        const struct dsdb_schema *schema = dsdb_get_schema(ldb);
        const struct dsdb_class *class;
        int i, j, ret;
+       /* Must remove any existing attribute, or else confusion reins */
+       ldb_msg_remove_attr(msg, attrName);
        ret = ldb_msg_add_empty(msg, attrName, 0, &allowedAttributes);
        if (ret != LDB_SUCCESS) {
                return ret;
@@ -182,6 +184,8 @@
        const struct dsdb_schema *schema = dsdb_get_schema(ldb);
        const struct dsdb_class *class;
        int i, j, ret;
+       /* Must remove any existing attribute, or else confusion reins */
+       ldb_msg_remove_attr(msg, attrName);
        ret = ldb_msg_add_empty(msg, attrName, 0, &allowedClasses);
        if (ret != LDB_SUCCESS) {
                return ret;