From: Joao Eduardo Luis <joao@suse.de>
Date: Mon, 7 Sep 2015 13:12:19 +0000 (+0100)
Subject: mon: MonitorDBStore: make get_next_key() work properly
X-Git-Tag: v9.1.0~185^2~1
X-Git-Url: http://git.apps.os.sepia.ceph.com/?a=commitdiff_plain;h=70d31082fd3dc8c7857994104577f1a3631c678c;p=ceph.git

mon: MonitorDBStore: make get_next_key() work properly

We introduced a significant bug with 2cc7aee, when we fixed issue #11786.
Although that patch would fix the problem described in #11786, we
managed to not increment the iterator upon returning the current key.
This would have the iterator iterating over the same key, forever and
ever.

Signed-off-by: Joao Eduardo Luis <joao@suse.de>
---

diff --git a/src/mon/MonitorDBStore.h b/src/mon/MonitorDBStore.h
index b9aab3094ef5b..1a6f419e515a3 100644
--- a/src/mon/MonitorDBStore.h
+++ b/src/mon/MonitorDBStore.h
@@ -461,8 +461,10 @@ class MonitorDBStore
 
       for (; iter->valid(); iter->next()) {
         pair<string,string> r = iter->raw_key();
-        if (sync_prefixes.count(r.first) > 0)
+        if (sync_prefixes.count(r.first) > 0) {
+          iter->next();
           return r;
+        }
       }
       return pair<string,string>();
     }