From 62320616ff8795ff18c8f49029d81f12558c10ed Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Tue, 28 Aug 2007 05:43:26 +0000 Subject: [PATCH] r24731: Remove unused code - if we hit these error conditions, then we are dead anyway, and a segfault would leave us with more infomation. Andrew Bartlett --- source/dsdb/samdb/ldb_modules/extended_dn.c | 5 ----- source/dsdb/samdb/ldb_modules/kludge_acl.c | 9 --------- source/dsdb/samdb/ldb_modules/local_password.c | 18 ------------------ source/dsdb/samdb/ldb_modules/objectclass.c | 5 ----- source/dsdb/samdb/ldb_modules/partition.c | 16 ---------------- source/dsdb/samdb/ldb_modules/password_hash.c | 10 ---------- source/dsdb/samdb/ldb_modules/rootdse.c | 5 ----- source/dsdb/samdb/ldb_modules/schema.c | 6 ------ source/dsdb/samdb/ldb_modules/show_deleted.c | 8 -------- 9 files changed, 82 deletions(-) diff --git a/source/dsdb/samdb/ldb_modules/extended_dn.c b/source/dsdb/samdb/ldb_modules/extended_dn.c index 0a3227a9120..6f32d22d26d 100644 --- a/source/dsdb/samdb/ldb_modules/extended_dn.c +++ b/source/dsdb/samdb/ldb_modules/extended_dn.c @@ -181,11 +181,6 @@ static int extended_callback(struct ldb_context *ldb, void *context, struct ldb_ { struct extended_context *ac; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - goto error; - } - ac = talloc_get_type(context, struct extended_context); if (ares->type == LDB_REPLY_ENTRY) { diff --git a/source/dsdb/samdb/ldb_modules/kludge_acl.c b/source/dsdb/samdb/ldb_modules/kludge_acl.c index d26c8e34857..2d6d8a2d801 100644 --- a/source/dsdb/samdb/ldb_modules/kludge_acl.c +++ b/source/dsdb/samdb/ldb_modules/kludge_acl.c @@ -236,11 +236,6 @@ static int kludge_acl_callback(struct ldb_context *ldb, void *context, struct ld struct kludge_private_data *data; int i, ret; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - goto error; - } - ac = talloc_get_type(context, struct kludge_acl_context); data = talloc_get_type(ac->module->private_data, struct kludge_private_data); @@ -296,10 +291,6 @@ static int kludge_acl_callback(struct ldb_context *ldb, void *context, struct ld } return ac->up_callback(ldb, ac->up_context, ares); - -error: - talloc_free(ares); - return LDB_ERR_OPERATIONS_ERROR; } static int kludge_acl_search(struct ldb_module *module, struct ldb_request *req) diff --git a/source/dsdb/samdb/ldb_modules/local_password.c b/source/dsdb/samdb/ldb_modules/local_password.c index 97909c06b52..573d2e27b96 100644 --- a/source/dsdb/samdb/ldb_modules/local_password.c +++ b/source/dsdb/samdb/ldb_modules/local_password.c @@ -355,11 +355,6 @@ static int get_self_callback(struct ldb_context *ldb, void *context, struct ldb_ { struct lpdb_context *ac; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - return LDB_ERR_OPERATIONS_ERROR; - } - ac = talloc_get_type(context, struct lpdb_context); /* we are interested only in the single reply (base search) we receive here */ @@ -461,11 +456,6 @@ static int lpdb_local_search_callback(struct ldb_context *ldb, void *context, st { struct lpdb_local_search_context *local_context; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - return LDB_ERR_OPERATIONS_ERROR; - } - local_context = talloc_get_type(context, struct lpdb_local_search_context); /* we are interested only in the single reply (base search) we receive here */ @@ -529,11 +519,6 @@ static int lpdb_remote_search_callback(struct ldb_context *ldb, void *context, s { struct lpdb_context *ac; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - goto error; - } - ac = talloc_get_type(context, struct lpdb_context); if (ares->type == LDB_REPLY_ENTRY) { @@ -610,9 +595,6 @@ static int lpdb_remote_search_callback(struct ldb_context *ldb, void *context, s } else { return ac->orig_req->callback(ldb, ac->orig_req->context, ares); } -error: - talloc_free(ares); - return LDB_ERR_OPERATIONS_ERROR; } /* Search for passwords and other attributes. The passwords are diff --git a/source/dsdb/samdb/ldb_modules/objectclass.c b/source/dsdb/samdb/ldb_modules/objectclass.c index f7b2da9b839..04cf8efdb27 100644 --- a/source/dsdb/samdb/ldb_modules/objectclass.c +++ b/source/dsdb/samdb/ldb_modules/objectclass.c @@ -480,11 +480,6 @@ static int get_self_callback(struct ldb_context *ldb, void *context, struct ldb_ { struct oc_context *ac; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - return LDB_ERR_OPERATIONS_ERROR; - } - ac = talloc_get_type(context, struct oc_context); /* we are interested only in the single reply (base search) we receive here */ diff --git a/source/dsdb/samdb/ldb_modules/partition.c b/source/dsdb/samdb/ldb_modules/partition.c index ae739f9b00f..0675f38c566 100644 --- a/source/dsdb/samdb/ldb_modules/partition.c +++ b/source/dsdb/samdb/ldb_modules/partition.c @@ -45,7 +45,6 @@ struct partition_private_data { struct partition_context { struct ldb_module *module; - struct ldb_handle *handle; struct ldb_request *orig_req; struct ldb_request **down_req; @@ -76,7 +75,6 @@ static struct partition_context *partition_init_handle(struct ldb_request *req, h->private_data = ac; ac->module = module; - ac->handle = h; ac->orig_req = req; req->handle = h; @@ -126,11 +124,6 @@ static int partition_search_callback(struct ldb_context *ldb, void *context, str { struct partition_context *ac; - if (!context || !ares) { - ldb_set_errstring(ldb, "partition_search_callback: NULL Context or Result in 'search' callback"); - goto error; - } - ac = talloc_get_type(context, struct partition_context); if (ares->type == LDB_REPLY_ENTRY) { @@ -144,9 +137,6 @@ static int partition_search_callback(struct ldb_context *ldb, void *context, str return LDB_SUCCESS; } } -error: - talloc_free(ares); - return LDB_ERR_OPERATIONS_ERROR; } /* @@ -156,11 +146,6 @@ static int partition_other_callback(struct ldb_context *ldb, void *context, stru { struct partition_context *ac; - if (!context) { - ldb_set_errstring(ldb, "partition_other_callback: NULL Context in 'other' callback"); - goto error; - } - ac = talloc_get_type(context, struct partition_context); if (!ac->orig_req->callback) { @@ -179,7 +164,6 @@ static int partition_other_callback(struct ldb_context *ldb, void *context, stru return LDB_SUCCESS; } ldb_set_errstring(ldb, "partition_other_callback: Unknown reply type, only supports START_TLS"); -error: talloc_free(ares); return LDB_ERR_OPERATIONS_ERROR; } diff --git a/source/dsdb/samdb/ldb_modules/password_hash.c b/source/dsdb/samdb/ldb_modules/password_hash.c index 718e0480af8..61e90024395 100644 --- a/source/dsdb/samdb/ldb_modules/password_hash.c +++ b/source/dsdb/samdb/ldb_modules/password_hash.c @@ -1168,11 +1168,6 @@ static int get_domain_data_callback(struct ldb_context *ldb, void *context, stru { struct ph_context *ac; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - return LDB_ERR_OPERATIONS_ERROR; - } - ac = talloc_get_type(context, struct ph_context); /* we are interested only in the single reply (base search) we receive here */ @@ -1629,11 +1624,6 @@ static int get_self_callback(struct ldb_context *ldb, void *context, struct ldb_ { struct ph_context *ac; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - return LDB_ERR_OPERATIONS_ERROR; - } - ac = talloc_get_type(context, struct ph_context); /* we are interested only in the single reply (base search) we receive here */ diff --git a/source/dsdb/samdb/ldb_modules/rootdse.c b/source/dsdb/samdb/ldb_modules/rootdse.c index 0afc2dfb8ed..a808d674e29 100644 --- a/source/dsdb/samdb/ldb_modules/rootdse.c +++ b/source/dsdb/samdb/ldb_modules/rootdse.c @@ -226,11 +226,6 @@ static int rootdse_callback(struct ldb_context *ldb, void *context, struct ldb_r { struct rootdse_context *ac; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - goto error; - } - ac = talloc_get_type(context, struct rootdse_context); if (ares->type == LDB_REPLY_ENTRY) { diff --git a/source/dsdb/samdb/ldb_modules/schema.c b/source/dsdb/samdb/ldb_modules/schema.c index 2de5e892ed1..21e93e17158 100644 --- a/source/dsdb/samdb/ldb_modules/schema.c +++ b/source/dsdb/samdb/ldb_modules/schema.c @@ -523,11 +523,6 @@ static int schema_add_check_parent(struct ldb_context *ldb, void *context, struc { struct schema_context *sctx; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - return LDB_ERR_OPERATIONS_ERROR; - } - sctx = talloc_get_type(context, struct schema_context); /* we are interested only in the single reply (base search) we receive here */ @@ -883,7 +878,6 @@ static int schema_add_build_down_req(struct schema_context *sctx) { struct schema_class_dlist *temp; struct ldb_message *msg; - char *oc; int ret; sctx->down_req = talloc(sctx, struct ldb_request); diff --git a/source/dsdb/samdb/ldb_modules/show_deleted.c b/source/dsdb/samdb/ldb_modules/show_deleted.c index cbc9d50b06a..19fa63fb6e9 100644 --- a/source/dsdb/samdb/ldb_modules/show_deleted.c +++ b/source/dsdb/samdb/ldb_modules/show_deleted.c @@ -52,11 +52,6 @@ static int show_deleted_search_callback(struct ldb_context *ldb, void *context, { struct show_deleted_search_request *ar; - if (!context || !ares) { - ldb_set_errstring(ldb, "NULL Context or Result in callback"); - goto error; - } - ar = talloc_get_type(context, struct show_deleted_search_request); if (ares->type == LDB_REPLY_ENTRY) { @@ -78,9 +73,6 @@ static int show_deleted_search_callback(struct ldb_context *ldb, void *context, skip_deleted: talloc_free(ares); return LDB_SUCCESS; -error: - talloc_free(ares); - return LDB_ERR_OPERATIONS_ERROR; } static int show_deleted_search(struct ldb_module *module, struct ldb_request *req)