From: David Zafman <dzafman@redhat.com>
Date: Wed, 27 Apr 2016 01:16:22 +0000 (-0700)
Subject: rados: Balance format sections in same do_get_inconsistent_cmd()
X-Git-Tag: v10.2.3~142^2~6
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=80f0dd348f32ff6e1f98ca29ea2ed38b5b7f5854;p=ceph.git

rados: Balance format sections in same do_get_inconsistent_cmd()

Signed-off-by: David Zafman <dzafman@redhat.com>
(cherry picked from commit de4681b80a5d24e72bbc7ac6dfee4775987fd834)
---

diff --git a/src/tools/rados/rados.cc b/src/tools/rados/rados.cc
index 428e526cb7ab..8c346eab0a31 100644
--- a/src/tools/rados/rados.cc
+++ b/src/tools/rados/rados.cc
@@ -1321,7 +1321,6 @@ static void dump_inconsistent(const inconsistent_obj_t& inc,
     f.close_section();
   }
   f.close_section();
-  f.close_section();
 }
 
 static void dump_inconsistent(const inconsistent_snapset_t& inc,
@@ -1349,7 +1348,6 @@ static void dump_inconsistent(const inconsistent_snapset_t& inc,
     }
     f.close_section();
   }
-  f.close_section();
 }
 
 // dispatch the call by type
@@ -1425,6 +1423,7 @@ static int do_get_inconsistent_cmd(const std::vector<const char*> &nargs,
     for (auto& inc : items) {
       formatter.open_object_section("inconsistent");
       dump_inconsistent(inc, formatter);
+      formatter.close_section();
     }
     if (items.size() < max_item_num) {
       formatter.close_section();