mon/MDSMonitor.cc: fix error handling in 'mds getmap'

Fix for:

CID 1297889 (#1 of 1): Unused value (UNUSED_VALUE)
 assigned_value: Assigning value -2 to r here, but that
 stored value is overwritten before it can be used.

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>

diff --git a/src/mon/MDSMonitor.cc b/src/mon/MDSMonitor.cc
index d588e14c304d77f236553da576ee5174b5163cee..b04ea2d9ef0fbeb5f29ce83b3f0e7f9dc5f645e4 100644 (file)
--- a/src/mon/MDSMonitor.cc
+++ b/src/mon/MDSMonitor.cc
@@ -830,12 +830,13 @@ bool MDSMonitor::preprocess_command(MonOpRequestRef op)
        mm.decode(b);
        mm.encode(rdata, m->get_connection()->get_features());
        ss << "got mdsmap epoch " << mm.get_epoch();
+       r = 0;
       }
     } else {
       mdsmap.encode(rdata, m->get_connection()->get_features());
       ss << "got mdsmap epoch " << mdsmap.get_epoch();
+      r = 0;
     }
-    r = 0;
   } else if (prefix == "mds tell") {
     string whostr;
     cmd_getval(g_ceph_context, cmdmap, "who", whostr);