From 3bc21143552b35698c9916c67494336de8964d2a Mon Sep 17 00:00:00 2001 From: Josh Durgin Date: Tue, 29 Jan 2013 14:22:15 -0800 Subject: [PATCH] ObjectCacher: fix flush_set when no flushing is needed C_GatherBuilder takes ownership of the Context we pass it. Deleting it in flush_set after constructing the C_GatherBuilder results in a double delete. Fixes: #3946 Signed-off-by: Josh Durgin Reviewed-by: Sam Lang --- src/osdc/ObjectCacher.cc | 1 - 1 file changed, 1 deletion(-) diff --git a/src/osdc/ObjectCacher.cc b/src/osdc/ObjectCacher.cc index bf83cfa49c9c8..265a806dbb231 100644 --- a/src/osdc/ObjectCacher.cc +++ b/src/osdc/ObjectCacher.cc @@ -1495,7 +1495,6 @@ bool ObjectCacher::flush_set(ObjectSet *oset, Context *onfinish) if (safe) { ldout(cct, 10) << "flush_set " << oset << " has no dirty|tx bhs" << dendl; - delete onfinish; return true; } return false; -- 2.39.5