From: Patrick Donnelly <pdonnell@redhat.com>
Date: Mon, 10 Oct 2016 22:13:33 +0000 (-0400)
Subject: mds: use reference to avoid copy
X-Git-Tag: v10.2.6~36^2~6^2~2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=89653e98a385df27d986d6a686d1a32b978d2792;p=ceph.git

mds: use reference to avoid copy

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 8934e95d75acadc1a82347272ac18175751a92de)
---

diff --git a/src/mds/FSMap.cc b/src/mds/FSMap.cc
index eb3dd74f0169..0d39d9cc6bae 100644
--- a/src/mds/FSMap.cc
+++ b/src/mds/FSMap.cc
@@ -499,7 +499,7 @@ int FSMap::parse_filesystem(
   std::string ns_err;
   fs_cluster_id_t fscid = strict_strtol(ns_str.c_str(), 10, &ns_err);
   if (!ns_err.empty() || filesystems.count(fscid) == 0) {
-    for (auto fs : filesystems) {
+    for (auto &fs : filesystems) {
       if (fs.second->mds_map.fs_name == ns_str) {
         *result = std::const_pointer_cast<const Filesystem>(fs.second);
         return 0;
@@ -732,7 +732,7 @@ void FSMap::erase(mds_gid_t who, epoch_t blacklist_epoch)
     standby_daemons.erase(who);
     standby_epochs.erase(who);
   } else {
-    auto fs = filesystems.at(mds_roles.at(who));
+    auto &fs = filesystems.at(mds_roles.at(who));
     const auto &info = fs->mds_map.mds_info.at(who);
     if (info.state != MDSMap::STATE_STANDBY_REPLAY) {
       if (info.state == MDSMap::STATE_CREATING) {