From bc663a979837f84fbd6d5f3bed1444d170ce7abb Mon Sep 17 00:00:00 2001 From: Andrew Bartlett Date: Thu, 16 May 2019 15:12:55 +1200 Subject: [PATCH] dsdb/partition: Move in_transaction decrement to end of partition_del_trans() It makes no sense for this to be mid-function. Signed-off-by: Andrew Bartlett Reviewed-by: Gary Lockyer --- source4/dsdb/samdb/ldb_modules/partition.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/source4/dsdb/samdb/ldb_modules/partition.c b/source4/dsdb/samdb/ldb_modules/partition.c index 49bdeb04fa5..dca4b7f7993 100644 --- a/source4/dsdb/samdb/ldb_modules/partition.c +++ b/source4/dsdb/samdb/ldb_modules/partition.c @@ -1193,12 +1193,6 @@ int partition_del_trans(struct ldb_module *module) } } - if (data->in_transaction == 0) { - DEBUG(0,("partition del transaction mismatch\n")); - return ldb_operr(ldb_module_get_ctx(module)); - } - data->in_transaction--; - if (ldb_module_flags(ldb_module_get_ctx(module)) & LDB_FLG_ENABLE_TRACING) { ldb_debug(ldb_module_get_ctx(module), LDB_DEBUG_TRACE, "partition_del_trans() -> (metadata partition)"); @@ -1213,6 +1207,12 @@ int partition_del_trans(struct ldb_module *module) final_ret = ret; } + if (data->in_transaction == 0) { + DEBUG(0,("partition del transaction mismatch\n")); + return ldb_operr(ldb_module_get_ctx(module)); + } + data->in_transaction--; + return final_ret; } -- 2.47.3