From 67950c27e473ebf8f7f81ef0ef92d2bd7931622a Mon Sep 17 00:00:00 2001 From: Andrew Tridgell Date: Tue, 16 Feb 2010 14:23:21 +1100 Subject: s4-dsdb: move dsdb_request_add_controls() into dsdb/common/util.c This will be used to allow the flag based ldb functions to work on both a ldb or a module, thus saving a lot of specialist functions. --- source4/dsdb/samdb/ldb_modules/extended_dn_store.c | 2 +- source4/dsdb/samdb/ldb_modules/samldb.c | 2 +- source4/dsdb/samdb/ldb_modules/util.c | 80 ++-------------------- source4/dsdb/samdb/ldb_modules/util.h | 14 ++-- 4 files changed, 13 insertions(+), 85 deletions(-) (limited to 'source4/dsdb/samdb') diff --git a/source4/dsdb/samdb/ldb_modules/extended_dn_store.c b/source4/dsdb/samdb/ldb_modules/extended_dn_store.c index 0b9a105be6..3c4c171c19 100644 --- a/source4/dsdb/samdb/ldb_modules/extended_dn_store.c +++ b/source4/dsdb/samdb/ldb_modules/extended_dn_store.c @@ -275,7 +275,7 @@ static int extended_store_replace(struct extended_dn_context *ac, return ret; } - ret = dsdb_request_add_controls(ac->module, os->search_req, + ret = dsdb_request_add_controls(os->search_req, DSDB_SEARCH_SHOW_DELETED|DSDB_SEARCH_SHOW_DN_IN_STORAGE_FORMAT); if (ret != LDB_SUCCESS) { talloc_free(os); diff --git a/source4/dsdb/samdb/ldb_modules/samldb.c b/source4/dsdb/samdb/ldb_modules/samldb.c index ccf76aaef2..9d79776e66 100644 --- a/source4/dsdb/samdb/ldb_modules/samldb.c +++ b/source4/dsdb/samdb/ldb_modules/samldb.c @@ -680,7 +680,7 @@ static int samldb_find_for_defaultObjectCategory(struct samldb_ctx *ac) return ret; } - ret = dsdb_request_add_controls(ac->module, req, + ret = dsdb_request_add_controls(req, DSDB_SEARCH_SHOW_DN_IN_STORAGE_FORMAT); if (ret != LDB_SUCCESS) { return ret; diff --git a/source4/dsdb/samdb/ldb_modules/util.c b/source4/dsdb/samdb/ldb_modules/util.c index 46252cb279..d3c58568ac 100644 --- a/source4/dsdb/samdb/ldb_modules/util.c +++ b/source4/dsdb/samdb/ldb_modules/util.c @@ -28,76 +28,6 @@ #include "util.h" #include "libcli/security/security.h" -/* - add a set of controls to a ldb_request structure based on a set of - flags. See util.h for a list of available flags - */ -int dsdb_request_add_controls(struct ldb_module *module, struct ldb_request *req, uint32_t dsdb_flags) -{ - int ret; - if (dsdb_flags & DSDB_SEARCH_SEARCH_ALL_PARTITIONS) { - struct ldb_search_options_control *options; - /* Using the phantom root control allows us to search all partitions */ - options = talloc(req, struct ldb_search_options_control); - if (options == NULL) { - ldb_module_oom(module); - return LDB_ERR_OPERATIONS_ERROR; - } - options->search_options = LDB_SEARCH_OPTION_PHANTOM_ROOT; - - ret = ldb_request_add_control(req, - LDB_CONTROL_SEARCH_OPTIONS_OID, - true, options); - if (ret != LDB_SUCCESS) { - return ret; - } - } - - if (dsdb_flags & DSDB_SEARCH_SHOW_DELETED) { - ret = ldb_request_add_control(req, LDB_CONTROL_SHOW_DELETED_OID, true, NULL); - if (ret != LDB_SUCCESS) { - return ret; - } - } - - if (dsdb_flags & DSDB_SEARCH_SHOW_DN_IN_STORAGE_FORMAT) { - ret = ldb_request_add_control(req, DSDB_CONTROL_DN_STORAGE_FORMAT_OID, true, NULL); - if (ret != LDB_SUCCESS) { - return ret; - } - } - - if (dsdb_flags & DSDB_SEARCH_SHOW_EXTENDED_DN) { - struct ldb_extended_dn_control *extended_ctrl = talloc(req, struct ldb_extended_dn_control); - if (!extended_ctrl) { - ldb_module_oom(module); - return LDB_ERR_OPERATIONS_ERROR; - } - extended_ctrl->type = 1; - - ret = ldb_request_add_control(req, LDB_CONTROL_EXTENDED_DN_OID, true, extended_ctrl); - if (ret != LDB_SUCCESS) { - return ret; - } - } - - if (dsdb_flags & DSDB_SEARCH_REVEAL_INTERNALS) { - ret = ldb_request_add_control(req, LDB_CONTROL_REVEAL_INTERNALS, false, NULL); - if (ret != LDB_SUCCESS) { - return ret; - } - } - - if (dsdb_flags & DSDB_MODIFY_RELAX) { - ret = ldb_request_add_control(req, LDB_CONTROL_RELAX_OID, false, NULL); - if (ret != LDB_SUCCESS) { - return ret; - } - } - - return LDB_SUCCESS; -} - /* search for attrs on one DN, in the modules below */ @@ -134,7 +64,7 @@ int dsdb_module_search_dn(struct ldb_module *module, return ret; } - ret = dsdb_request_add_controls(module, req, dsdb_flags); + ret = dsdb_request_add_controls(req, dsdb_flags); if (ret != LDB_SUCCESS) { talloc_free(tmp_ctx); return ret; @@ -206,7 +136,7 @@ int dsdb_module_search(struct ldb_module *module, return ret; } - ret = dsdb_request_add_controls(module, req, dsdb_flags); + ret = dsdb_request_add_controls(req, dsdb_flags); if (ret != LDB_SUCCESS) { talloc_free(tmp_ctx); return ret; @@ -325,7 +255,7 @@ int dsdb_module_modify(struct ldb_module *module, return ret; } - ret = dsdb_request_add_controls(module, mod_req, dsdb_flags); + ret = dsdb_request_add_controls(mod_req, dsdb_flags); if (ret != LDB_SUCCESS) { talloc_free(tmp_ctx); return ret; @@ -375,7 +305,7 @@ int dsdb_module_rename(struct ldb_module *module, return ret; } - ret = dsdb_request_add_controls(module, req, dsdb_flags); + ret = dsdb_request_add_controls(req, dsdb_flags); if (ret != LDB_SUCCESS) { talloc_free(tmp_ctx); return ret; @@ -422,7 +352,7 @@ int dsdb_module_add(struct ldb_module *module, return ret; } - ret = dsdb_request_add_controls(module, req, dsdb_flags); + ret = dsdb_request_add_controls(req, dsdb_flags); if (ret != LDB_SUCCESS) { talloc_free(tmp_ctx); return ret; diff --git a/source4/dsdb/samdb/ldb_modules/util.h b/source4/dsdb/samdb/ldb_modules/util.h index 53ed9bd48e..9ba2e33f56 100644 --- a/source4/dsdb/samdb/ldb_modules/util.h +++ b/source4/dsdb/samdb/ldb_modules/util.h @@ -26,12 +26,10 @@ struct dsdb_attribute; struct dsdb_fsmo_extended_op; #include "dsdb/samdb/ldb_modules/util_proto.h" +#include "dsdb/common/util.h" + +/* extend the dsdb_request_add_controls() flags for module + specific functions */ +#define DSDB_FLAG_OWN_MODULE 0x00400000 +#define DSDB_FLAG_TOP_MODULE 0x00800000 -#define DSDB_SEARCH_SEARCH_ALL_PARTITIONS 0x0001 -#define DSDB_SEARCH_SHOW_DELETED 0x0002 -#define DSDB_SEARCH_SHOW_DN_IN_STORAGE_FORMAT 0x0004 -#define DSDB_SEARCH_REVEAL_INTERNALS 0x0008 -#define DSDB_SEARCH_SHOW_EXTENDED_DN 0x0010 -#define DSDB_MODIFY_RELAX 0x0020 -#define DSDB_FLAG_OWN_MODULE 0x0040 -#define DSDB_FLAG_TOP_MODULE 0x0080 -- cgit