librbd/cache/pwl: WriteLogOperationSet::cell can be garbage

The pointer is never initialized but gets printed by operator<<.
Luckily outside of that it's unused.

Fixes: https://tracker.ceph.com/issues/74971
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
(cherry picked from commit bffa11487cb7d68c0aa39994f50fbc3b4b00e415)

diff --git a/src/librbd/cache/pwl/LogOperation.cc b/src/librbd/cache/pwl/LogOperation.cc
index c051f561b65c95b0c370573f5f8f76b5601f5818..0b9553b6babe051e71ba28fdde11cca4b31c3b15 100644 (file)
--- a/src/librbd/cache/pwl/LogOperation.cc
+++ b/src/librbd/cache/pwl/LogOperation.cc
@@ -268,8 +268,7 @@ WriteLogOperationSet::~WriteLogOperationSet() { }
 
 std::ostream &operator<<(std::ostream &os,
                          const WriteLogOperationSet &s) {
-  os << "cell=" << (void*)s.cell
-     << ", extent_ops_appending=" << s.extent_ops_appending
+  os << "extent_ops_appending=" << s.extent_ops_appending
      << ", extent_ops_persist=" << s.extent_ops_persist;
   return os;
 }

diff --git a/src/librbd/cache/pwl/LogOperation.h b/src/librbd/cache/pwl/LogOperation.h
index 15befe05f255d2a91977e8a26ef8da7563f70338..1496e18b7bcebedef79d0c439dc638dd0116a523 100644 (file)
--- a/src/librbd/cache/pwl/LogOperation.h
+++ b/src/librbd/cache/pwl/LogOperation.h
@@ -169,7 +169,6 @@ private:
   Context *m_on_finish;
 public:
   bool persist_on_flush;
-  BlockGuardCell *cell;
   C_Gather *extent_ops_appending;
   Context *on_ops_appending;
   C_Gather *extent_ops_persist;