s4-dsdb Remove mem_ctx argument from dsdb_module_find_dsheuristics().
authorAndrew Bartlett <abartlet@samba.org>
Thu, 25 Nov 2010 05:12:39 +0000 (16:12 +1100)
committerAndrew Bartlett <abartlet@samba.org>
Thu, 25 Nov 2010 21:50:42 +0000 (08:50 +1100)
A function that does not return memory should not take a memory context.

Andrew Bartlett

source4/dsdb/samdb/ldb_modules/acl_read.c
source4/dsdb/samdb/ldb_modules/util.c

index 401e0dc318f1c90b24dbf9b79e38c6ab48852577..3a0f6f34eb2983a9d5cff7316129def260e2e3a0 100644 (file)
@@ -226,7 +226,7 @@ static int aclread_search(struct ldb_module *module, struct ldb_request *req)
 
        session_info = (struct auth_session_info *)ldb_get_opaque(ldb, "sessionInfo");
        if (session_info && security_token_is_anonymous(session_info->security_token)) {
-               block_anonymous = dsdb_block_anonymous_ops(module, req);
+               block_anonymous = dsdb_block_anonymous_ops(module);
                if (block_anonymous) {
                        return ldb_error(ldb, LDB_ERR_OPERATIONS_ERROR,
                                         "This request is not allowed to an anonymous connection.");
index 01d7347d3eb9e756d3992da1131fbfe320cd9ade..9608b061c4e464f688c49e15a2fcf0db66c30eed 100644 (file)
@@ -1080,10 +1080,9 @@ const struct ldb_val *dsdb_module_find_dsheuristics(struct ldb_module *module,
        return NULL;
 }
 
-bool dsdb_block_anonymous_ops(struct ldb_module *module,
-                             TALLOC_CTX *mem_ctx)
+bool dsdb_block_anonymous_ops(struct ldb_module *module)
 {
-       TALLOC_CTX *tmp_ctx = talloc_new(mem_ctx);
+       TALLOC_CTX *tmp_ctx = talloc_new(module);
        bool result;
        const struct ldb_val *hr_val = dsdb_module_find_dsheuristics(module,
                                                                     tmp_ctx);