From: Joao Eduardo Luis <joao@suse.de>
Date: Wed, 27 May 2015 21:28:49 +0000 (+0100)
Subject: mon: MonitorDBStore: get_next_key() only if prefix matches
X-Git-Tag: v9.0.3~19^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=2cc7aee1abe40453093306c8fef2312b650dff5d;p=ceph.git

mon: MonitorDBStore: get_next_key() only if prefix matches

get_next_key() had a bug in which we would always return the first key
from the iterator, regardless of whether its prefix had been specified
to the iterator.

Fixes: #11786

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

diff --git a/src/mon/MonitorDBStore.h b/src/mon/MonitorDBStore.h
index d00d3607960b..9feb266c49ee 100644
--- a/src/mon/MonitorDBStore.h
+++ b/src/mon/MonitorDBStore.h
@@ -448,11 +448,13 @@ class MonitorDBStore
 
     virtual pair<string,string> get_next_key() {
       assert(iter->valid());
-      pair<string,string> r = iter->raw_key();
-      do {
-	iter->next();
-      } while (iter->valid() && sync_prefixes.count(iter->raw_key().first) == 0);
-      return r;
+
+      for (; iter->valid(); iter->next()) {
+        pair<string,string> r = iter->raw_key();
+        if (sync_prefixes.count(r.first) > 0)
+          return r;
+      }
+      return pair<string,string>();
     }
 
     virtual bool _is_valid() {