crimson/osd/pg: tolerate NULL PG::pglog_based_recovery_op when

cancelling pglog based recovery

The finish of pglog based recovery and the pg turning Clean are two
asynchronous events, other events like DeferRecovery should tolerate
this case.

Fixes: https://tracker.ceph.com/issues/72844
Signed-off-by: Xuehan Xu <xuxuehan@qianxin.com>

diff --git a/src/crimson/osd/pg.cc b/src/crimson/osd/pg.cc
index c7ae18d2f252bd0785104e2918fd67625450697c..b9bcfebd022faf70028e89e574825531668a26d8 100644 (file)
--- a/src/crimson/osd/pg.cc
+++ b/src/crimson/osd/pg.cc
@@ -1744,9 +1744,10 @@ void PG::reset_pglog_based_recovery_op() {
 }
 
 void PG::cancel_pglog_based_recovery_op() {
-  ceph_assert(pglog_based_recovery_op);
-  pglog_based_recovery_op->cancel();
-  reset_pglog_based_recovery_op();
+  if (pglog_based_recovery_op) {
+    pglog_based_recovery_op->cancel();
+    reset_pglog_based_recovery_op();
+  }
 }
 
 void PG::C_PG_FinishRecovery::finish(int r) {