librados: only shutdown objecter after it's initialized

The objecter is only initialized once the RadosClient state is
CONNECTED from the perspective of a RadosClient::shutdown()
caller. Error paths in RadosClient::connect() may call shutdown while
still in the CONNECTING state.

Signed-off-by: Josh Durgin <josh.durgin@dreamhost.com>
Reviewed-by: Samuel Just <samuel.just@dreamhost.com>

1 files changed, 2 insertions, 2 deletions

diff --git a/src/librados.cc b/src/librados.cc
index 7262b759c94..0e558cdafe3 100644
--- a/src/librados.cc
+++ b/src/librados.cc
@@ -1015,11 +1015,11 @@ void librados::RadosClient::shutdown()
     lock.Unlock();
     return;
   }
-  state = DISCONNECTED;
   monclient.shutdown();
-  if (objecter)
+  if (objecter && state == CONNECTED)
     objecter->shutdown();
   timer.shutdown();
+  state = DISCONNECTED;
   lock.Unlock();
   if (messenger) {
     messenger->shutdown();