curvebs/client: mds address should not be empty

Signed-off-by: Hanqing Wu <[email protected]>

Author: wu-hanqing

src/client/client_config.cpp

@@ -184,6 +184,12 @@ int ClientConfig::Init(const std::string& configpath) {
 
     std::vector<std::string> mdsAddr;
     common::SplitString(metaAddr, ",", &mdsAddr);
+    if (mdsAddr.empty()) {
+        LOG(ERROR) << "mds.listen.addr seems invalid or empty, `" << metaAddr
+                   << "`'";
+        return -1;
+    }
+
     fileServiceOption_.metaServerOpt.rpcRetryOpt.addrs.assign(mdsAddr.begin(),
                                                         mdsAddr.end());
     for (auto& addr : fileServiceOption_.metaServerOpt.rpcRetryOpt.addrs) {

src/client/mds_client.cpp

@@ -191,6 +191,9 @@ int RPCExcutorRetryPolicy::GetNextMDSIndex(bool needChangeMDS,
 
 int RPCExcutorRetryPolicy::ExcuteTask(int mdsindex, uint64_t rpcTimeOutMS,
                                       RPCFunc task) {
+    assert(mdsindex >= 0 &&
+           mdsindex < static_cast<int>(retryOpt_.addrs.size()));
+
     const std::string &mdsaddr = retryOpt_.addrs[mdsindex];
 
     brpc::Channel channel;