rgw: handle bad/unknown checksum case in cksum::Factory

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>

diff --git a/src/rgw/rgw_cksum_pipe.cc b/src/rgw/rgw_cksum_pipe.cc
index 0bec8d341af078bbc3eca7f20c90c1126b35c599..32b93755f5d8fdb8ec86ca6322b7ad0bb04440d1 100644 (file)
--- a/src/rgw/rgw_cksum_pipe.cc
+++ b/src/rgw/rgw_cksum_pipe.cc
@@ -43,9 +43,14 @@ namespace rgw::putobj {
     if (algo_header.first) {
       if (algo_header.second) {
        auto cksum_type = cksum::parse_cksum_type(algo_header.second);
-       return
-         std::make_unique<RGWPutObj_Cksum>(
-                                   next, cksum_type, std::move(algo_header));
+       /* unknown checksum type in header */
+       if (cksum_type != rgw::cksum::Type::none) {
+         return
+           std::make_unique<RGWPutObj_Cksum>(
+                              next, cksum_type, std::move(algo_header));
+       } else {
+         return std::unique_ptr<RGWPutObj_Cksum>();
+       }
       }
       /* malformed checksum algorithm header(s) */
       throw rgw::io::Exception(EINVAL, std::system_category());