r17514: Simplify the way to set ldb errors and add another
[abartlet/samba.git/.git] / source4 / lib / ldb / modules / sort.c
index a37442b41f7dc27825bbd486743d49f41cdbe41b..0ae16d08ab9a265b703764f283bd5f0bb216315b 100644 (file)
@@ -43,11 +43,10 @@ struct opaque {
        int result;
 };
 
-struct sort_async_context {
+struct sort_context {
        struct ldb_module *module;
        void *up_context;
-       int (*up_callback)(struct ldb_context *, void *, struct ldb_async_result *);
-       int timeout;
+       int (*up_callback)(struct ldb_context *, void *, struct ldb_reply *);
 
        char *attributeName;
        char *orderingRule;
@@ -64,35 +63,36 @@ struct sort_async_context {
        int sort_result;
 };
 
-static struct ldb_async_handle *init_handle(void *mem_ctx, struct ldb_module *module,
+static struct ldb_handle *init_handle(void *mem_ctx, struct ldb_module *module,
                                            void *context,
-                                           int (*callback)(struct ldb_context *, void *, struct ldb_async_result *),
-                                           int timeout)
+                                           int (*callback)(struct ldb_context *, void *, struct ldb_reply *))
 {
-       struct sort_async_context *ac;
-       struct ldb_async_handle *h;
+       struct sort_context *ac;
+       struct ldb_handle *h;
 
-       h = talloc_zero(mem_ctx, struct ldb_async_handle);
+       h = talloc_zero(mem_ctx, struct ldb_handle);
        if (h == NULL) {
-               ldb_set_errstring(module->ldb, talloc_asprintf(module, "Out of Memory"));
+               ldb_set_errstring(module->ldb, "Out of Memory");
                return NULL;
        }
 
        h->module = module;
 
-       ac = talloc_zero(h, struct sort_async_context);
+       ac = talloc_zero(h, struct sort_context);
        if (ac == NULL) {
-               ldb_set_errstring(module->ldb, talloc_asprintf(module, "Out of Memory"));
+               ldb_set_errstring(module->ldb, "Out of Memory");
                talloc_free(h);
                return NULL;
        }
 
        h->private_data = (void *)ac;
 
+       h->state = LDB_ASYNC_INIT;
+       h->status = LDB_SUCCESS;
+
        ac->module = module;
        ac->up_context = context;
        ac->up_callback = callback;
-       ac->timeout = timeout;
 
        return h;
 }
@@ -141,34 +141,7 @@ static int build_response(void *mem_ctx, struct ldb_control ***ctrls, int result
 
 static int sort_compare(struct ldb_message **msg1, struct ldb_message **msg2, void *opaque)
 {
-       struct opaque *data = (struct opaque *)opaque;
-       struct ldb_message_element *el1, *el2;
-
-       if (data->result != 0) {
-               /* an error occurred previously,
-                * let's exit the sorting by returning always 0 */
-               return 0;
-       }
-
-       el1 = ldb_msg_find_element(*msg1, data->attribute);
-       el2 = ldb_msg_find_element(*msg2, data->attribute);
-
-       if (!el1 || !el2) {
-               /* the attribute was not found return and
-                * set an error */
-               data->result = 53;
-               return 0;
-       }
-
-       if (data->reverse)
-               return data->h->comparison_fn(data->ldb, data, &el2->values[0], &el1->values[0]);
-
-       return data->h->comparison_fn(data->ldb, data, &el1->values[0], &el2->values[0]);
-}
-
-static int sort_compare_async(struct ldb_message **msg1, struct ldb_message **msg2, void *opaque)
-{
-       struct sort_async_context *ac = talloc_get_type(opaque, struct sort_async_context);
+       struct sort_context *ac = talloc_get_type(opaque, struct sort_context);
        struct ldb_message_element *el1, *el2;
 
        if (ac->sort_result != 0) {
@@ -193,99 +166,16 @@ static int sort_compare_async(struct ldb_message **msg1, struct ldb_message **ms
        return ac->h->comparison_fn(ac->module->ldb, ac, &el1->values[0], &el2->values[0]);
 }
 
-/* search */
-static int server_sort_search(struct ldb_module *module, struct ldb_control *control, struct ldb_request *req)
-{
-       struct ldb_result *sort_result = NULL;
-       struct ldb_control **saved_controls;
-       struct ldb_server_sort_control **sort_ctrls;
-       int ret, result = 0;
-       int do_sort = 1;
-
-       sort_ctrls = talloc_get_type(control->data, struct ldb_server_sort_control *);
-       if (!sort_ctrls) {
-               return LDB_ERR_PROTOCOL_ERROR;
-       }
-
-       /* FIXME: we do not support more than one attribute for sorting right now */
-       /* FIXME: we need to check if the attribute type exist or return an error */
-       if (sort_ctrls[1] != NULL)
-               do_sort = 0;
-               
-       if (!do_sort && control->critical) {
-               sort_result = talloc_zero(req, struct ldb_result);
-               if (!sort_result)
-                       return LDB_ERR_OPERATIONS_ERROR;
-
-               req->op.search.res = sort_result;
-       
-               /* 53 = unwilling to perform */
-               if ((ret = build_response(sort_result, &sort_result->controls, 53, "sort control is not complete yet")) != LDB_SUCCESS) {
-                       return ret;
-               }
-
-               return LDB_ERR_UNSUPPORTED_CRITICAL_EXTENSION;
-       }
-
-       /* save it locally and remove it from the list */
-       if (!save_controls(control, req, &saved_controls)) {
-               return LDB_ERR_OPERATIONS_ERROR;
-       }
-
-       ret = ldb_next_request(module, req);
-
-       if (req->controls) talloc_free(req->controls);
-       req->controls = saved_controls;
-
-       if (ret != LDB_SUCCESS) {
-               return ret;
-       }
-
-       /* SORT HERE */
-       if (do_sort) {
-               struct opaque *data;
-              
-               data = talloc(module, struct opaque);
-               if (!data)
-                       return LDB_ERR_OPERATIONS_ERROR;
-               
-               data->attribute = sort_ctrls[0]->attributeName;
-               data->reverse = sort_ctrls[0]->reverse;
-               data->ldb = module->ldb;
-               data->h = ldb_attrib_handler(data->ldb, data->attribute);
-               data->result = 0;
-               sort_result = req->op.search.res;
-
-               ldb_qsort(sort_result->msgs,
-                         sort_result->count,
-                         sizeof(struct ldb_message *),
-                         data,
-                         (ldb_qsort_cmp_fn_t)sort_compare);
-
-               result = data->result;
-
-               talloc_free(data);
-       } else {
-               result = 53;
-       }
-
-       if ((ret = build_response(sort_result, &sort_result->controls, result, "sort control is not complete yet")) != LDB_SUCCESS) {
-               return ret;
-       }
-
-       return LDB_SUCCESS;
-}
-
-static int server_sort_search_async_callback(struct ldb_context *ldb, void *context, struct ldb_async_result *ares)
+static int server_sort_search_callback(struct ldb_context *ldb, void *context, struct ldb_reply *ares)
 {
-       struct sort_async_context *ac = NULL;
+       struct sort_context *ac = NULL;
        
        if (!context || !ares) {
-               ldb_set_errstring(ldb, talloc_asprintf(ldb, "NULL Context or Result in callback"));
+               ldb_set_errstring(ldb, "NULL Context or Result in callback");
                goto error;
        }       
 
-               ac = talloc_get_type(context, struct sort_async_context);
+               ac = talloc_get_type(context, struct sort_context);
 
        if (ares->type == LDB_REPLY_ENTRY) {
                ac->msgs = talloc_realloc(ac, ac->msgs, struct ldb_message *, ac->num_msgs + 2);
@@ -336,26 +226,35 @@ error:
        return LDB_ERR_OPERATIONS_ERROR;
 }
 
-static int server_sort_search_async(struct ldb_module *module, struct ldb_control *control, struct ldb_request *req)
+static int server_sort_search(struct ldb_module *module, struct ldb_request *req)
 {
+       struct ldb_control *control;
        struct ldb_server_sort_control **sort_ctrls;
        struct ldb_control **saved_controls;
-       struct sort_async_context *ac;
-       struct ldb_async_handle *h;
+       struct sort_context *ac;
+       struct ldb_handle *h;
        int ret;
 
-       req->async.handle = NULL;
+       /* check if there's a paged request control */
+       control = get_control_from_list(req->controls, LDB_CONTROL_SERVER_SORT_OID);
+       if (control == NULL) {
+               /* not found go on */
+               return ldb_next_request(module, req);
+       }
+
+       req->handle = NULL;
 
-       if (!req->async.callback || !req->async.context) {
-               ldb_set_errstring(module->ldb, talloc_asprintf(module, "Async interface called with NULL callback function or NULL context"));
+       if (!req->callback || !req->context) {
+               ldb_set_errstring(module->ldb,
+                                 "Async interface called with NULL callback function or NULL context");
                return LDB_ERR_OPERATIONS_ERROR;
        }
        
-       h = init_handle(req, module, req->async.context, req->async.callback, req->async.timeout);
+       h = init_handle(req, module, req->context, req->callback);
        if (!h) {
                return LDB_ERR_OPERATIONS_ERROR;
        }
-       ac = talloc_get_type(h->private_data, struct sort_async_context);
+       ac = talloc_get_type(h->private_data, struct sort_context);
 
        sort_ctrls = talloc_get_type(control->data, struct ldb_server_sort_control *);
        if (!sort_ctrls) {
@@ -367,9 +266,9 @@ static int server_sort_search_async(struct ldb_module *module, struct ldb_contro
                
        if (sort_ctrls[1] != NULL) {
                if (control->critical) {
-                       struct ldb_async_result *ares;
+                       struct ldb_reply *ares;
 
-                       ares = talloc_zero(req, struct ldb_async_result);
+                       ares = talloc_zero(req, struct ldb_reply);
                        if (!ares)
                                return LDB_ERR_OPERATIONS_ERROR;
 
@@ -395,6 +294,8 @@ static int server_sort_search_async(struct ldb_module *module, struct ldb_contro
        ac->reverse = sort_ctrls[0]->reverse;
 
        ac->req = talloc(req, struct ldb_request);
+       if (!ac->req)
+               return LDB_ERR_OPERATIONS_ERROR;
 
        ac->req->operation = req->operation;
        ac->req->op.search.base = req->op.search.base;
@@ -410,59 +311,32 @@ static int server_sort_search_async(struct ldb_module *module, struct ldb_contro
                return LDB_ERR_OPERATIONS_ERROR;
        }
 
-       ac->req->creds = req->creds;
-
-       ac->req->async.context = ac;
-       ac->req->async.callback = server_sort_search_async_callback;
-       ac->req->async.timeout = req->async.timeout;
+       ac->req->context = ac;
+       ac->req->callback = server_sort_search_callback;
+       ldb_set_timeout_from_prev_req(module->ldb, req, ac->req);
 
-       req->async.handle = h;
+       req->handle = h;
 
        return ldb_next_request(module, ac->req);
 }
 
-static int server_sort(struct ldb_module *module, struct ldb_request *req)
+static int server_sort_results(struct ldb_handle *handle)
 {
-       struct ldb_control *control;
-
-       /* check if there's a paged request control */
-       control = get_control_from_list(req->controls, LDB_CONTROL_SERVER_SORT_OID);
-       if (control == NULL) {
-               /* not found go on */
-               return ldb_next_request(module, req);
-       }
-
-       switch (req->operation) {
-
-       case LDB_REQ_SEARCH:
-               return server_sort_search(module, control, req);
-
-       case LDB_ASYNC_SEARCH:
-               return server_sort_search_async(module, control, req);
-
-       default:
-               return LDB_ERR_PROTOCOL_ERROR;
-
-       }
-}
-
-static int server_sort_async_results(struct ldb_async_handle *handle)
-{
-       struct sort_async_context *ac;
-       struct ldb_async_result *ares;
+       struct sort_context *ac;
+       struct ldb_reply *ares;
        int i, ret;
 
-       ac = talloc_get_type(handle->private_data, struct sort_async_context);
+       ac = talloc_get_type(handle->private_data, struct sort_context);
 
        ac->h = ldb_attrib_handler(ac->module->ldb, ac->attributeName);
        ac->sort_result = 0;
 
        ldb_qsort(ac->msgs, ac->num_msgs,
                  sizeof(struct ldb_message *),
-                 ac, (ldb_qsort_cmp_fn_t)sort_compare_async);
+                 ac, (ldb_qsort_cmp_fn_t)sort_compare);
 
        for (i = 0; i < ac->num_msgs; i++) {
-               ares = talloc_zero(ac, struct ldb_async_result);
+               ares = talloc_zero(ac, struct ldb_reply);
                if (!ares) {
                        handle->status = LDB_ERR_OPERATIONS_ERROR;
                        return handle->status;
@@ -478,7 +352,7 @@ static int server_sort_async_results(struct ldb_async_handle *handle)
        }
 
        for (i = 0; i < ac->num_refs; i++) {
-               ares = talloc_zero(ac, struct ldb_async_result);
+               ares = talloc_zero(ac, struct ldb_reply);
                if (!ares) {
                        handle->status = LDB_ERR_OPERATIONS_ERROR;
                        return handle->status;
@@ -493,7 +367,7 @@ static int server_sort_async_results(struct ldb_async_handle *handle)
                }
        }
 
-       ares = talloc_zero(ac, struct ldb_async_result);
+       ares = talloc_zero(ac, struct ldb_reply);
        if (!ares) {
                handle->status = LDB_ERR_OPERATIONS_ERROR;
                return handle->status;
@@ -514,33 +388,33 @@ static int server_sort_async_results(struct ldb_async_handle *handle)
        return LDB_SUCCESS;
 }
 
-static int server_sort_async_wait(struct ldb_async_handle *handle, enum ldb_async_wait_type type)
+static int server_sort_wait(struct ldb_handle *handle, enum ldb_wait_type type)
 {
-       struct sort_async_context *ac;
+       struct sort_context *ac;
        int ret;
     
        if (!handle || !handle->private_data) {
                return LDB_ERR_OPERATIONS_ERROR;
        }
 
-       ac = talloc_get_type(handle->private_data, struct sort_async_context);
+       ac = talloc_get_type(handle->private_data, struct sort_context);
 
-       ret = ldb_async_wait(handle->module->ldb, ac->req->async.handle, type);
+       ret = ldb_wait(ac->req->handle, type);
 
        if (ret != LDB_SUCCESS) {
                handle->status = ret;
                return ret;
        }
                
-       handle->state = ac->req->async.handle->state;
-       handle->status = ac->req->async.handle->status;
+       handle->state = ac->req->handle->state;
+       handle->status = ac->req->handle->status;
 
        if (handle->status != LDB_SUCCESS) {
                return handle->status;
        }
 
        if (handle->state == LDB_ASYNC_DONE) {
-               ret = server_sort_async_results(handle);
+               ret = server_sort_results(handle);
        }
 
        return ret;
@@ -548,26 +422,33 @@ static int server_sort_async_wait(struct ldb_async_handle *handle, enum ldb_asyn
 
 static int server_sort_init(struct ldb_module *module)
 {
-       struct ldb_request request;
+       struct ldb_request *req;
        int ret;
 
-       request.operation = LDB_REQ_REGISTER;
-       request.op.reg.oid = LDB_CONTROL_SERVER_SORT_OID;
-       request.controls = NULL;
+       req = talloc(module, struct ldb_request);
+       if (req == NULL) {
+               return LDB_ERR_OPERATIONS_ERROR;
+       }
+
+       req->operation = LDB_REQ_REGISTER_CONTROL;
+       req->op.reg_control.oid = LDB_CONTROL_SERVER_SORT_OID;
+       req->controls = NULL;
 
-       ret = ldb_request(module->ldb, &request);
+       ret = ldb_request(module->ldb, req);
        if (ret != LDB_SUCCESS) {
                ldb_debug(module->ldb, LDB_DEBUG_ERROR, "server_sort: Unable to register control with rootdse!\n");
+               talloc_free(req);
                return LDB_ERR_OTHER;
        }
 
+       talloc_free(req);
        return ldb_next_init(module);
 }
 
 static const struct ldb_module_ops server_sort_ops = {
        .name              = "server_sort",
-       .request           = server_sort,
-       .async_wait        = server_sort_async_wait,
+       .search            = server_sort_search,
+       .wait              = server_sort_wait,
        .init_context      = server_sort_init
 };