From 470742d87dcd385d5cb0a3cd21a0b66897798598 Mon Sep 17 00:00:00 2001 From: Greg Farnum Date: Mon, 17 Oct 2011 10:34:17 -0700 Subject: [PATCH] cls_rgw: move stat update code after error checks in complete_op Signed-off-by: Greg Farnum --- src/cls_rgw.cc | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/src/cls_rgw.cc b/src/cls_rgw.cc index 3135f1e8e23d0..20ecd398f6374 100644 --- a/src/cls_rgw.cc +++ b/src/cls_rgw.cc @@ -214,13 +214,6 @@ int rgw_bucket_complete_op(cls_method_context_t hctx, bufferlist *in, bufferlist bufferlist::iterator cur_iter = current_entry.begin(); ::decode(entry, cur_iter); CLS_LOG("rgw_bucket_complete_op(): existing entry: epoch=%lld\n", entry.epoch); - - if (entry.exists) { - struct rgw_bucket_category_stats& stats = header.stats[entry.meta.category]; - stats.num_entries--; - stats.total_size -= entry.meta.size; - stats.total_size_rounded -= get_rounded_size(entry.meta.size); - } } if (op.tag.size()) { @@ -237,6 +230,13 @@ int rgw_bucket_complete_op(cls_method_context_t hctx, bufferlist *in, bufferlist return 0; } + if (entry.exists) { + struct rgw_bucket_category_stats& stats = header.stats[entry.meta.category]; + stats.num_entries--; + stats.total_size -= entry.meta.size; + stats.total_size_rounded -= get_rounded_size(entry.meta.size); + } + bufferlist op_bl; switch (op.op) { -- 2.39.5