Merge pull request #14677: jewel: mds: C_MDSInternalNoop::complete do…

…esn't free itself Reviewed-by: Nathan Cutler <ncutler@suse.com>
ceph · Apr 27, 2017 · 0fab783 · 0fab783
2 parents 0a9ddc6 + 7347f11
commit 0fab783
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/src/mds/MDSContext.h b/src/mds/MDSContext.h
@@ -114,12 +114,12 @@ class MDSIOContextWrapper : public MDSIOContextBase
 /**
  * No-op for callers expecting MDSInternalContextBase
  */
-class C_MDSInternalNoop : public MDSInternalContextBase
+class C_MDSInternalNoop final : public MDSInternalContextBase
 {
   virtual MDSRank* get_mds() {assert(0);}
 public:
   void finish(int r) {}
-  void complete(int r) {}
+  void complete(int r) { delete this; }
 };