rgw: fix inverted MFA check in DeleteMultiObj

A versioned delete carries a non-empty version-id, but the MFA guard
flagged the empty case: versioned deletes bypassed MFA while plain
deletes wrongly required it. Test !instance.empty().

Fixes: https://tracker.ceph.com/issues/77869
Signed-off-by: Sun Yuechi <sunyuechi@iscas.ac.cn>

diff --git a/src/rgw/rgw_op.cc b/src/rgw/rgw_op.cc
index bfcb0ea6df1f0647d5b944fbcc47b0099e2ee55f..43c2fc309636b1fd082f4b5163188b0a7215eac4 100644 (file)
--- a/src/rgw/rgw_op.cc
+++ b/src/rgw/rgw_op.cc
@@ -8358,7 +8358,7 @@ void RGWDeleteMultiObj::execute(optional_yield y)
     bool has_versioned = false;
     for (auto object : multi_delete->objects) {
       const string& instance = object.get_version_id();
-      if (instance.empty()) {
+      if (!instance.empty()) {
         has_versioned = true;
         break;
       }