From dfbe4f9cb078bad9a0244ce433a9728f8bd6b912 Mon Sep 17 00:00:00 2001 From: Cory Snyder Date: Wed, 14 Jun 2023 14:01:43 +0000 Subject: [PATCH] rgw: apply_olh_log should clear_olh before trimming olh log When pending_removal is true and the log is trimmed prior to clearing olh, it allows for the possibility that the olh is not cleared (e.g. due to new pending xattrs on the olh obj), but the olh log entries are removed. Signed-off-by: Cory Snyder --- src/rgw/driver/rados/rgw_rados.cc | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) diff --git a/src/rgw/driver/rados/rgw_rados.cc b/src/rgw/driver/rados/rgw_rados.cc index e3f0054dd61..1de5ce56f6e 100644 --- a/src/rgw/driver/rados/rgw_rados.cc +++ b/src/rgw/driver/rados/rgw_rados.cc @@ -7544,17 +7544,21 @@ int RGWRados::apply_olh_log(const DoutPrefixProvider *dpp, return r; } + if (need_to_remove) { + string olh_tag(state.olh_tag.c_str(), state.olh_tag.length()); + r = clear_olh(dpp, obj_ctx, obj, bucket_info, ref, olh_tag, last_ver, y); + if (r < 0 && r != -ECANCELED) { + ldpp_dout(dpp, 0) << "ERROR: could not clear olh, r=" << r << dendl; + return r; + } + } + r = bucket_index_trim_olh_log(dpp, bucket_info, state, obj, last_ver, y); - if (r < 0) { + if (r < 0 && r != -ECANCELED) { ldpp_dout(dpp, 0) << "ERROR: could not trim olh log, r=" << r << dendl; return r; } - if (need_to_remove) { - string olh_tag(state.olh_tag.c_str(), state.olh_tag.length()); - clear_olh(dpp, obj_ctx, obj, bucket_info, ref, olh_tag, last_ver, y); - } - return 0; } -- 2.39.5