Applied, thanks!
On Mon, 24 Sep 2012, Yan, Zheng wrote:
> 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>
> ---
> src/mds/SessionMap.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> 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
