From 52f2f9e55d296e439f4f80f93affb1e01b971829 Mon Sep 17 00:00:00 2001 From: Greg Farnum Date: Tue, 31 Aug 2021 20:18:49 +0000 Subject: [PATCH] crimson: osd: do not capture std::error_code by reference I can't find any reason to assume it has a longer lifetime than the future we're creating has. Signed-off-by: Greg Farnum --- src/crimson/osd/pg.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/crimson/osd/pg.cc b/src/crimson/osd/pg.cc index fbebbc4b8158b..704b6bf563a55 100644 --- a/src/crimson/osd/pg.cc +++ b/src/crimson/osd/pg.cc @@ -716,7 +716,7 @@ PG::do_osd_ops_execute( [rollbacker, failure_func_ptr] (const std::error_code& e) mutable { return rollbacker.rollback_obc_if_modified(e).then_interruptible( - [&e, failure_func_ptr] { + [e, failure_func_ptr] { return (*failure_func_ptr)(e); }); }) @@ -728,7 +728,7 @@ PG::do_osd_ops_execute( return PG::do_osd_ops_iertr::make_ready_future>( seastar::now(), rollbacker.rollback_obc_if_modified(e).then_interruptible( - [&e, failure_func_ptr] { + [e, failure_func_ptr] { return (*failure_func_ptr)(e); })); })); -- 2.39.5