common: assert non-null ptrs in OnFinisher

Callers should not be constructing C_OnFinisher
if they have no context to call back.

Signed-off-by: John Spray <john.spray@redhat.com>

diff --git a/src/common/Finisher.h b/src/common/Finisher.h
index 610470eb4e1268739d65ad8129636d4f3cea3000..229342079d8dbb1ef210634efe001efd05eb550e 100644 (file)
--- a/src/common/Finisher.h
+++ b/src/common/Finisher.h
@@ -122,7 +122,10 @@ class C_OnFinisher : public Context {
   Context *con;
   Finisher *fin;
 public:
-  C_OnFinisher(Context *c, Finisher *f) : con(c), fin(f) {}
+  C_OnFinisher(Context *c, Finisher *f) : con(c), fin(f) {
+    assert(fin != NULL);
+    assert(con != NULL);
+  }
   void finish(int r) {
     fin->queue(con, r);
   }