From 4b6688ee257ba3441fc406e12555cf48d5a090a6 Mon Sep 17 00:00:00 2001 From: Sage Weil Date: Thu, 18 Mar 2010 10:20:10 -0700 Subject: [PATCH] mds: session is always defined in handle_client_reconnect --- src/mds/Server.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mds/Server.cc b/src/mds/Server.cc index ec3eaa53ca2ca..ff4004a68ef2e 100644 --- a/src/mds/Server.cc +++ b/src/mds/Server.cc @@ -480,6 +480,7 @@ void Server::handle_client_reconnect(MClientReconnect *m) dout(7) << "handle_client_reconnect " << m->get_source() << dendl; int from = m->get_source().num(); Session *session = get_session(m); + assert(session); if (!mds->is_reconnect() && mds->get_want_state() == CEPH_MDS_STATE_RECONNECT) { dout(10) << " we're almost in reconnect state (mdsmap delivery race?); waiting" << dendl; @@ -492,7 +493,7 @@ void Server::handle_client_reconnect(MClientReconnect *m) delay -= reconnect_start; dout(10) << " reconnect_start " << reconnect_start << " delay " << delay << dendl; - if (!mds->is_reconnect() || !session || session->is_closed()) { + if (!mds->is_reconnect() || session->is_closed()) { if (!mds->is_reconnect()) { // XXX maybe in the future we can do better than this? dout(1) << " no longer in reconnect state, ignoring reconnect, sending close" << dendl; -- 2.39.5