Merge pull request ceph#65159 from Suyashd999/fix-check-source-header

yuvalif · web-flow · commit 3f36e67aa040 · 2025-10-22T10:42:46.000+03:00
rgw: Check if `HTTP_X_AMZ_COPY_SOURCE` header is empty
diff --git a/src/rgw/rgw_op.cc b/src/rgw/rgw_op.cc
@@ -5753,6 +5753,9 @@ bool RGWCopyObj::parse_copy_location(const std::string_view& url_src,
     params_str = url_src.substr(pos + 1);
   }
 
+  if (name_str.empty()) {
+    return false;
+  }
   if (name_str[0] == '/') // trim leading slash
     name_str.remove_prefix(1);
 

Original file line number	Diff line number	Diff line change
`@@ -5753,6 +5753,9 @@ bool RGWCopyObj::parse_copy_location(const std::string_view& url_src,`
`5753`	`5753`	`params_str = url_src.substr(pos + 1);`
`5754`	`5754`	`}`
`5755`	`5755`
	`5756`	`+ if (name_str.empty()) {`
	`5757`	`+ return false;`
	`5758`	`+ }`
`5756`	`5759`	`if (name_str[0] == '/') // trim leading slash`
`5757`	`5760`	`name_str.remove_prefix(1);`
`5758`	`5761`