]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commit
rgw/multisite: Fix use-after-move in retry logic in logbacking
authorAdam Emerson <aemerson@redhat.com>
Tue, 4 Jun 2024 03:17:33 +0000 (23:17 -0400)
committerAdam Emerson <aemerson@redhat.com>
Fri, 10 Jan 2025 21:21:39 +0000 (16:21 -0500)
commitd29d8b356b5c704a08f0b4bf654a5d30e1d7a626
tree17799a7e8634bb995acecadbc52fdfb5a4e863f6
parent8a28a0aebfa0e95f142cb00d67403340afc840a5
rgw/multisite: Fix use-after-move in retry logic in logbacking

Fitting a retry loop around a call that took the lock changed the
logic to potentially have the lock be used after moving.

Thanks to Suyash Dongre suyashd999@gmail.com for finding it.

This patch also fixes a lock being held across an I/O operation.

Fixes: https://tracker.ceph.com/issues/66340
Signed-off-by: Adam Emerson <aemerson@redhat.com>
(cherry picked from commit 80cf2ca3b44246ad8f2e333182b3c8b1835a71f7)

Fixes: https://tracker.ceph.com/issues/68054
Signed-off-by: Adam Emerson <aemerson@redhat.com>
src/rgw/driver/rados/rgw_log_backing.cc