diff --git a/cpp/arcticdb/storage/s3/s3_client_wrapper.cpp b/cpp/arcticdb/storage/s3/s3_client_wrapper.cpp
index d9315a8492..5b270fd035 100644
--- a/cpp/arcticdb/storage/s3/s3_client_wrapper.cpp
+++ b/cpp/arcticdb/storage/s3/s3_client_wrapper.cpp
@@ -102,7 +102,7 @@ folly::Future<S3Result<Segment>> S3ClientTestWrapper::get_object_async(
 
 S3Result<std::monostate> S3ClientTestWrapper::put_object(
         const std::string &s3_object_name,
-        Segment &&segment,
+        Segment &segment,
         const std::string &bucket_name,
         PutHeader header) {
     auto maybe_error = has_failure_trigger(bucket_name);
@@ -110,7 +110,7 @@ S3Result<std::monostate> S3ClientTestWrapper::put_object(
         return {*maybe_error};
     }
 
-    return actual_client_->put_object(s3_object_name, std::move(segment), bucket_name, header);
+    return actual_client_->put_object(s3_object_name, segment, bucket_name, header);
 }
 
 S3Result<DeleteOutput> S3ClientTestWrapper::delete_objects(
diff --git a/cpp/arcticdb/storage/s3/s3_client_wrapper.hpp b/cpp/arcticdb/storage/s3/s3_client_wrapper.hpp
index 370d573c96..4639036d85 100644
--- a/cpp/arcticdb/storage/s3/s3_client_wrapper.hpp
+++ b/cpp/arcticdb/storage/s3/s3_client_wrapper.hpp
@@ -50,7 +50,7 @@ class S3ClientTestWrapper : public S3ClientInterface {
 
     S3Result<std::monostate> put_object(
         const std::string& s3_object_name,
-        Segment&& segment,
+        Segment& segment,
         const std::string& bucket_name,
         PutHeader header = PutHeader::NONE) override;