s4:samldb LDB module - remove "type" parameter of "samldb_fill_object"
authorMatthias Dieter Wallnöfer <mdw@samba.org>
Wed, 6 Oct 2010 09:37:28 +0000 (11:37 +0200)
committerMatthias Dieter Wallnöfer <mdw@sn-devel-104.sn.samba.org>
Wed, 6 Oct 2010 10:20:45 +0000 (10:20 +0000)
It's a bit redundant given that we have the "type" variable on "ac".

Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Wed Oct  6 10:20:45 UTC 2010 on sn-devel-104

source4/dsdb/samdb/ldb_modules/samldb.c

index f3a9e08e7277bd6fa6940fe925f91c7c057af8f2..c67f2d06d679c6edcff64b7a446e482dee85cdef 100644 (file)
@@ -612,7 +612,7 @@ static bool check_rodc_critical_attribute(struct ldb_message *msg)
 }
 
 
-static int samldb_fill_object(struct samldb_ctx *ac, const char *type)
+static int samldb_fill_object(struct samldb_ctx *ac)
 {
        struct ldb_context *ldb;
        struct loadparm_context *lp_ctx;
@@ -624,7 +624,6 @@ static int samldb_fill_object(struct samldb_ctx *ac, const char *type)
        ldb = ldb_module_get_ctx(ac->module);
 
        /* Add informations for the different account types */
-       ac->type = type;
        if (strcmp(ac->type, "user") == 0) {
                ret = samdb_find_or_add_attribute(ldb, ac->msg,
                        "userAccountControl", "546");
@@ -1103,12 +1102,14 @@ static int samldb_add(struct ldb_module *module, struct ldb_request *req)
 
        if (samdb_find_attribute(ldb, ac->msg,
                                 "objectclass", "user") != NULL) {
-               return samldb_fill_object(ac, "user");
+               ac->type = "user";
+               return samldb_fill_object(ac);
        }
 
        if (samdb_find_attribute(ldb, ac->msg,
                                 "objectclass", "group") != NULL) {
-               return samldb_fill_object(ac, "group");
+               ac->type = "group";
+               return samldb_fill_object(ac);
        }
 
        /* perhaps a foreignSecurityPrincipal? */
@@ -1126,7 +1127,8 @@ static int samldb_add(struct ldb_module *module, struct ldb_request *req)
                        return ret;
                }
 
-               return samldb_fill_object(ac, "classSchema");
+               ac->type = "classSchema";
+               return samldb_fill_object(ac);
        }
 
        if (samdb_find_attribute(ldb, ac->msg,
@@ -1137,7 +1139,8 @@ static int samldb_add(struct ldb_module *module, struct ldb_request *req)
                        return ret;
                }
 
-               return samldb_fill_object(ac, "attributeSchema");
+               ac->type = "attributeSchema";
+               return samldb_fill_object(ac);
        }
 
        talloc_free(ac);