From: "Yan, Zheng" <zheng.z.yan@xxxxxxxxx> is_any_state() should return false when the by_state list is empty. Signed-off-by: Yan, Zheng <zheng.z.yan@xxxxxxxxx> --- ceph-object-corpus | 2 +- src/mds/SessionMap.h | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ceph-object-corpus b/ceph-object-corpus index 7f23ee1..4d64c37 160000 --- a/ceph-object-corpus +++ b/ceph-object-corpus @@ -1 +1 @@ -Subproject commit 7f23ee1127101d311ec0c50adb719367a0789fad +Subproject commit 4d64c3751101141a34beadf74c3e9f013bce550c diff --git a/src/mds/SessionMap.h b/src/mds/SessionMap.h index fcd0b47..6bca361 100644 --- a/src/mds/SessionMap.h +++ b/src/mds/SessionMap.h @@ -248,7 +248,7 @@ public: bool is_any_state(int state) { map<int,xlist<Session*>* >::iterator p = by_state.find(state); - if (p == by_state.end()) + if (p == by_state.end() || p->second->empty()) return false; return true; } -- 1.7.11.4 -- To unsubscribe from this list: send the line "unsubscribe ceph-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html