mds: don't modify filepath when printing

filepath::depth() may call parse_bits(), which modifies mutable member
'bits'. dump_ops_in_flight asok command prints filepath without holding
mds_lock. So multiple threads may call parse_bits() at the same time.

Fixes: http://tracker.ceph.com/issues/26894
Signed-off-by: "Yan, Zheng" <zyan@redhat.com>
(cherry picked from commit 0f057feac379fb49a1ed42948b489ceefccc15b8)

diff --git a/src/include/filepath.h b/src/include/filepath.h
index 8439b3fc985412fbb5a48511af79af75276133b1..ff4b749ba26ea4d830661764e90f484746adb807 100644 (file)
--- a/src/include/filepath.h
+++ b/src/include/filepath.h
@@ -226,7 +226,7 @@ inline ostream& operator<<(ostream& out, const filepath& path)
 {
   if (path.get_ino()) {
     out << '#' << path.get_ino();
-    if (path.depth())
+    if (path.length())
       out << '/';
   }
   return out << path.get_path();