rgw: Remove logically dead code

Signed-off-by: Vedansh Bhartia <vedanshbhartia@gmail.com>

diff --git a/src/rgw/driver/rados/rgw_reshard.cc b/src/rgw/driver/rados/rgw_reshard.cc
index 31eff5312df4812e2d70fe30bb0400fa1fc5cc82..4369bc8a05c8bcf8b4cff8741b676c35fb675f28 100644 (file)
--- a/src/rgw/driver/rados/rgw_reshard.cc
+++ b/src/rgw/driver/rados/rgw_reshard.cc
@@ -1303,9 +1303,6 @@ int RGWReshard::process_single_logshard(int logshard_num, const DoutPrefixProvid
 
     for(auto& entry: entries) { // logshard entries
       process_entry(entry, max_entries, dpp, y);
-      if (ret < 0) {
-        return ret;
-      }
 
       Clock::time_point now = Clock::now();
       if (logshard_lock.should_renew(now)) {

diff --git a/src/rgw/rgw_sal_dbstore.cc b/src/rgw/rgw_sal_dbstore.cc
index df6330552e758f48c828840cd7130768f60391eb..9886bd5b590962727e62abb3a728f3457cf25902 100644 (file)
--- a/src/rgw/rgw_sal_dbstore.cc
+++ b/src/rgw/rgw_sal_dbstore.cc
@@ -1329,7 +1329,6 @@ namespace rgw::sal {
                        rgw_zone_set *zones_trace, bool *canceled,
                        optional_yield y)
   {
-    int ret = 0;
     /* XXX: same as AtomicWriter..consolidate code */
     parent_op.meta.mtime = mtime;
     parent_op.meta.delete_at = delete_at;
@@ -1341,9 +1340,6 @@ namespace rgw::sal {
     /* XXX: handle accounted size */
     accounted_size = total_data_size;
 
-    if (ret < 0)
-      return ret;
-
     RGWUploadPartInfo info;
     info.num = part_num;
     info.etag = etag;
@@ -1354,7 +1350,7 @@ namespace rgw::sal {
 
     DB::Object op_target(store->getDB(),
         meta_obj->get_bucket()->get_info(), meta_obj->get_obj());
-    ret = op_target.add_mp_part(dpp, info);
+    auto ret = op_target.add_mp_part(dpp, info);
     if (ret < 0) {
       return ret == -ENOENT ? -ERR_NO_SUCH_UPLOAD : ret;
     }