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 096e40388eafb9d131cda28ff408380465c7315b..40f9c26fa6e8fd60264d9fccdceeccda43077534 100644 (file)
--- a/src/include/filepath.h
+++ b/src/include/filepath.h
@@ -227,7 +227,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();