From b086dc47cdd56459258596eac381d9dc97c572f4 Mon Sep 17 00:00:00 2001 From: "Adam C. Emerson" Date: Tue, 8 Feb 2022 13:47:49 -0500 Subject: [PATCH] rgw: Fix data race in ChangeStatus Fixes: https://tracker.ceph.com/issues/54208 Signed-off-by: Adam C. Emerson (cherry picked from commit 27f5ba9e5f649d8767c8ab44d56404e0186f6fc1) Fixes: https://tracker.ceph.com/issues/54208 Signed-off-by: Adam C. Emerson --- src/rgw/rgw_datalog.cc | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/rgw/rgw_datalog.cc b/src/rgw/rgw_datalog.cc index 89e41471e11c..b52e52000774 100644 --- a/src/rgw/rgw_datalog.cc +++ b/src/rgw/rgw_datalog.cc @@ -566,13 +566,17 @@ void RGWDataChangesLog::register_renew(const rgw_bucket_shard& bs) void RGWDataChangesLog::update_renewed(const rgw_bucket_shard& bs, real_time expiration) { - std::scoped_lock l{lock}; + std::unique_lock l{lock}; ChangeStatusPtr status; _get_change(bs, status); + l.unlock(); + ldout(cct, 20) << "RGWDataChangesLog::update_renewd() bucket_name=" << bs.bucket.name << " shard_id=" << bs.shard_id << " expiration=" << expiration << dendl; + + std::unique_lock sl(status->lock); status->cur_expiration = expiration; } -- 2.47.3