---------- Forwarded message --------- From: &lt;scan-admin(a)coverity.com&gt; Date: Fri, Jan 12, 2024 at 4:15 PM Subject: New Defects reported by Coverity Scan for ceph To: &lt;kkeithle(a)redhat.com&gt; Hi, Please find the latest report on new defect(s) introduced to ceph found with Coverity Scan. 552 new defect(s) introduced to ceph found with Coverity Scan. 300 defect(s), reported by Coverity Scan earlier, were marked fixed in the recent build analyzed by Coverity Scan. New defect(s) Reported-by: Coverity Scan Showing 20 of 552 defect(s) ** CID 1585278: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /usr/include/c++/12/bits/stl_algobase.h: 1219 in std::__equal_aux<ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>, ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>>(T1, T1, T2)() ________________________________________________________________________________________________________ *** CID 1585278: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /usr/include/c++/12/bits/stl_algobase.h: 1219 in std::__equal_aux<ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>, ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>>(T1, T1, T2)() 1213 template<typename _II1, typename _II2> 1214 _GLIBCXX20_CONSTEXPR 1215 inline bool 1216 __equal_aux(_II1 __first1, _II1 __last1, _II2 __first2) 1217 { 1218 return std::__equal_aux1(std::__niter_base(__first1), could be moved instead. 1219 std::__niter_base(__last1), 1220 std::__niter_base(__first2)); 1221 } 1222 1223 template<typename _II1, typename _Seq1, typename _Cat1, typename _II2> 1224 bool ** CID 1585277: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_rados.cc: 9898 in RGWRados::cls_obj_usage_log_trim(const DoutPrefixProvider *, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, unsigned long, unsigned long, optional_yield)() ________________________________________________________________________________________________________ *** CID 1585277: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_rados.cc: 9898 in RGWRados::cls_obj_usage_log_trim(const DoutPrefixProvider *, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, unsigned long, unsigned long, optional_yield)() 9892 int RGWRados::cls_obj_usage_log_trim(const DoutPrefixProvider *dpp, const string& oid, const string& user, const string& bucket, 9893 uint64_t start_epoch, uint64_t end_epoch, optional_yield y) 9894 { 9895 rgw_raw_obj obj(svc.zone->get_zone_params().usage_log_pool, oid); 9896 9897 rgw_rados_ref ref; could be moved instead. 9898 int r = get_raw_obj_ref(dpp, obj, &ref); 9899 if (r < 0) { 9900 return r; 9901 } 9902 9903 r = cls_rgw_usage_log_trim_repeat(dpp, ref, user, bucket, start_epoch, end_epoch, y); ** CID 1585276: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/test/librbd/test_internal.cc: 1789 in librbd::TestInternal_SparsifyClone_Test::TestBody()() ________________________________________________________________________________________________________ *** CID 1585276: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/test/librbd/test_internal.cc: 1789 in librbd::TestInternal_SparsifyClone_Test::TestBody()() 1783 1784 bufferptr read_ptr(bl.length()); 1785 bufferlist read_bl; 1786 read_bl.push_back(read_ptr); 1787 1788 librbd::io::ReadResult read_result{&read_bl}; could be moved instead. 1789 ASSERT_EQ((ssize_t)read_bl.length(), 1790 api::Io<>::read(*ictx, (1 << ictx->order) * 10, read_bl.length(), 1791 librbd::io::ReadResult{read_result}, 0)); 1792 ASSERT_TRUE(bl.contents_equal(read_bl)); 1793 1794 std::string oid = ictx->get_object_name(0); ** CID 1585275: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_bucket_sync.cc: 584 in RGWBucketSyncFlowManager::init(const DoutPrefixProvider *, const rgw_sync_policy_info &)::[lambda(const rgw_zone_id &, std::optional<rgw_bucket>, const rgw_zone_id &, std::optional<rgw_bucket>) (instance 1)]::operator ()(const rgw_zone_id &, std::optional<rgw_bucket>, const rgw_zone_id &, std::optional<rgw_bucket>) const() ________________________________________________________________________________________________________ *** CID 1585275: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_bucket_sync.cc: 584 in RGWBucketSyncFlowManager::init(const DoutPrefixProvider *, const rgw_sync_policy_info &)::[lambda(const rgw_zone_id &, std::optional<rgw_bucket>, const rgw_zone_id &, std::optional<rgw_bucket>) (instance 1)]::operator ()(const rgw_zone_id &, std::optional<rgw_bucket>, const rgw_zone_id &, std::optional<rgw_bucket>) const() 578 const rgw_zone_id& dest_zone, 579 std::optional<rgw_bucket> dest_bucket) { 580 if (!parent) { 581 return true; 582 } 583 return parent->allowed_data_flow(source_zone, "allowed_data_flow" when it could be moved instead. 584 source_bucket, 585 dest_zone, 586 dest_bucket, 587 false); /* just check that it's not disabled */ 588 }); 589 } ** CID 1585274: (ORDER_REVERSAL) ________________________________________________________________________________________________________ *** CID 1585274: (ORDER_REVERSAL) /src/static/src/github/ceph/src/client/Client.cc: 12306 in Client::statfs(const char *, statvfs *, const UserPerm &)() 12300 // respect that one instead. 12301 ceph_assert(root != nullptr); 12302 InodeRef quota_root = root->quota.is_enabled(QUOTA_MAX_BYTES) ? root : get_quota_root(root.get(), perms, QUOTA_MAX_BYTES); 12303 12304 // get_quota_root should always give us something if client quotas are 12305 // enabled while holding lock "std::unique_lock<std::mutex>._M_device" (count: 2 / 5). 12306 ceph_assert(cct->_conf.get_val<bool>("client_quota") == false || quota_root != nullptr); 12307 12308 /* If bytes quota is set on a directory and conf option "client quota df" 12309 * is also set, available space = quota limit - used space. Else, 12310 * available space = total space - used space. */ 12311 if (quota_root && cct->_conf->client_quota_df && quota_root->quota.max_bytes) { /src/static/src/github/ceph/src/client/Client.cc: 12302 in Client::statfs(const char *, statvfs *, const UserPerm &)() 12296 stbuf->f_namemax = NAME_MAX; 12297 12298 // Usually quota_root will == root_ancestor, but if the mount root has no 12299 // quota but we can see a parent of it that does have a quota, we'll 12300 // respect that one instead. 12301 ceph_assert(root != nullptr); "ceph::common::ConfigProxy.lock" while holding lock "std::unique_lock<std::mutex>._M_device" (count: 2 / 5). 12302 InodeRef quota_root = root->quota.is_enabled(QUOTA_MAX_BYTES) ? root : get_quota_root(root.get(), perms, QUOTA_MAX_BYTES); 12303 12304 // get_quota_root should always give us something if client quotas are 12305 // enabled 12306 ceph_assert(cct->_conf.get_val<bool>("client_quota") == false || quota_root != nullptr); 12307 ** CID 1585273: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_sync.h: 294 in RGWMetaSyncStatusManager::read_master_log_shards_next(const DoutPrefixProvider *, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, std::map<int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::less<int>, std::allocator<std::pair<const int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>>>, std::map<int, rgw_mdlog_shard_data, std::less<int>, std::allocator<std::pair<const int, rgw_mdlog_shard_data>>>*)() ________________________________________________________________________________________________________ *** CID 1585273: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_sync.h: 294 in RGWMetaSyncStatusManager::read_master_log_shards_next(const DoutPrefixProvider *, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, std::map<int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::less<int>, std::allocator<std::pair<const int, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>>>, std::map<int, rgw_mdlog_shard_data, std::less<int>, std::allocator<std::pair<const int, rgw_mdlog_shard_data>>>*)() 288 return master_log.read_log_info(dpp, log_info); 289 } 290 int read_master_log_shards_info(const DoutPrefixProvider *dpp, const std::string& master_period, std::map<int, RGWMetadataLogInfo> *shards_info) { 291 return master_log.read_master_log_shards_info(dpp, master_period, shards_info); 292 } 293 int read_master_log_shards_next(const DoutPrefixProvider *dpp, const std::string& period, std::map<int, std::string> shard_markers, std::map<int, rgw_mdlog_shard_data> *result) { "read_master_log_shards_next" when it could be moved instead. 294 return master_log.read_master_log_shards_next(dpp, period, shard_markers, result); 295 } 296 297 int run(const DoutPrefixProvider *dpp, optional_yield y) { return master_log.run_sync(dpp, y); } 298 299 ** CID 1585272: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_sal_rados.cc: 536 in rgw::sal::RadosBucket::read_stats_async(const DoutPrefixProvider *, const rgw::bucket_index_layout_generation &, int, boost::intrusive_ptr<rgw::sal::ReadStatsCB>)() ________________________________________________________________________________________________________ *** CID 1585272: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_sal_rados.cc: 536 in rgw::sal::RadosBucket::read_stats_async(const DoutPrefixProvider *, const rgw::bucket_index_layout_generation &, int, boost::intrusive_ptr<rgw::sal::ReadStatsCB>)() 530 } 531 532 int RadosBucket::read_stats_async(const DoutPrefixProvider *dpp, 533 const bucket_index_layout_generation& idx_layout, 534 int shard_id, boost::intrusive_ptr<ReadStatsCB> ctx) 535 { when it could be moved instead. 536 return store->getRados()->get_bucket_stats_async(dpp, get_info(), idx_layout, shard_id, ctx); 537 } 538 539 int RadosBucket::sync_user_stats(const DoutPrefixProvider *dpp, optional_yield y, 540 RGWBucketEnt* ent) 541 { ** CID 1585271: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/tools/rbd/MirrorDaemonServiceInfo.cc: 104 in rbd::MirrorDaemonServiceInfo::get_mirror_service_dump()() ________________________________________________________________________________________________________ *** CID 1585271: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/tools/rbd/MirrorDaemonServiceInfo.cc: 104 in rbd::MirrorDaemonServiceInfo::get_mirror_service_dump()() 98 int MirrorDaemonServiceInfo::get_mirror_service_dump() { 99 librados::Rados rados(m_io_ctx); 100 std::string cmd = R"({"prefix": "service dump", "format": "json"})"; 101 bufferlist in_bl; 102 bufferlist out_bl; 103 could be moved instead. 104 int r = rados.mon_command(cmd, in_bl, &out_bl, nullptr); 105 if (r < 0) { 106 std::cerr << "rbd: failed to query services: " << cpp_strerror(r) 107 << std::endl; 108 return r; 109 } ** CID 1585270: Uninitialized variables (USE_AFTER_MOVE) /src/static/src/github/ceph/src/include/neorados/RADOS.hpp: 1697 in neorados::RADOS::enumerate_objects<boost::asio::redirect_error_t<boost::asio::use_awaitable_t<boost::asio::any_io_executor>>>(neorados::IOContext, neorados::Cursor, neorados::Cursor, unsigned int, ceph::buffer::v15_2_0::list, T1 &&)() ________________________________________________________________________________________________________ *** CID 1585270: Uninitialized variables (USE_AFTER_MOVE) /src/static/src/github/ceph/src/include/neorados/RADOS.hpp: 1697 in neorados::RADOS::enumerate_objects<boost::asio::redirect_error_t<boost::asio::use_awaitable_t<boost::asio::any_io_executor>>>(neorados::IOContext, neorados::Cursor, neorados::Cursor, unsigned int, ceph::buffer::v15_2_0::list, T1 &&)() 1691 using EnumerateComp = boost::asio::any_completion_handler<EnumerateSig>; 1692 template<boost::asio::completion_token_for<EnumerateSig> CompletionToken> 1693 auto enumerate_objects(IOContext ioc, Cursor begin, 1694 Cursor end, std::uint32_t max, 1695 ceph::buffer::list filter, 1696 CompletionToken&& token) { 1697 auto consigned = boost::asio::consign( 1698 std::forward<CompletionToken>(token), boost::asio::make_work_guard( 1699 boost::asio::get_associated_executor(token, get_executor()))); 1700 return boost::asio::async_initiate<decltype(consigned), EnumerateSig>( 1701 [ioc = std::move(ioc), begin = std::move(begin), end = std::move(end), 1702 max, filter = std::move(filter), this](auto&& handler) mutable { ** CID 1585269: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/tools/rbd/action/Perf.cc: 125 in rbd::action::perf::<unnamed>::query_iostats(librados::v14_2_0::Rados &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, rbd::action::perf::<unnamed>::StatDescriptor, std::vector<rbd::action::perf::<unnamed>::ImageStat, std::allocator<rbd::action::perf::<unnamed>::ImageStat>> *, std::basic_ostream<char, std::char_traits<char>>&)() ________________________________________________________________________________________________________ *** CID 1585269: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/tools/rbd/action/Perf.cc: 125 in rbd::action::perf::<unnamed>::query_iostats(librados::v14_2_0::Rados &, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, rbd::action::perf::<unnamed>::StatDescriptor, std::vector<rbd::action::perf::<unnamed>::ImageStat, std::allocator<rbd::action::perf::<unnamed>::ImageStat>> *, std::basic_ostream<char, std::char_traits<char>>&)() 119 "format": "json" 120 }")"; 121 122 bufferlist in_bl; 123 bufferlist out_bl; 124 std::string outs; could be moved instead. 125 int r = rados.mgr_command(cmd, in_bl, &out_bl, &outs); 126 if (r == -EOPNOTSUPP) { 127 err_os << "rbd: 'rbd_support' mgr module is not enabled." 128 << std::endl << std::endl 129 << "Use 'ceph mgr module enable rbd_support' to enable." 130 << std::endl; ** CID 1585268: Error handling issues (UNCAUGHT_EXCEPT) /src/static/src/github/ceph/src/osd/scrubber/scrub_machine.h: 195 in Scrub::IntervalChanged::~IntervalChanged()() ________________________________________________________________________________________________________ *** CID 1585268: Error handling issues (UNCAUGHT_EXCEPT) /src/static/src/github/ceph/src/osd/scrubber/scrub_machine.h: 195 in Scrub::IntervalChanged::~IntervalChanged()() 189 * This event is distinct from FullReset because replicas are always responsible 190 * for releasing any interval specific state (including but certainly not limited to 191 * scrub reservations) upon interval change, without coordination from the 192 * Primary. This event notifies the ScrubMachine that it can forget about 193 * such remote state. 194 */ but the exception specification "/*implicit*/noexcept" doesn't allow it to be thrown. This will result in a call to terminate(). 195 MEV(IntervalChanged) 196 197 /** 198 * stops the scrubbing session, and resets the scrubber. 199 * For a replica - aborts the handling of the current request. 200 * In both cases - a transition to the peering mode quiescent state (i.e. ** CID 1585267: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/osd/scrubber/scrub_machine.cc: 180 in Scrub::Session::Session(boost::statechart::state<Scrub::Session, Scrub::PrimaryActive, Scrub::ReservingReplicas, (boost::statechart::history_mode)0>::my_context)() ________________________________________________________________________________________________________ *** CID 1585267: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/osd/scrubber/scrub_machine.cc: 180 in Scrub::Session::Session(boost::statechart::state<Scrub::Session, Scrub::PrimaryActive, Scrub::ReservingReplicas, (boost::statechart::history_mode)0>::my_context)() 174 scrbr->clear_pgscrub_state(); 175 } 176 177 // ----------------------- Session ----------------------------------------- 178 179 Session::Session(my_context ctx) "boost::statechart::state<Scrub::Session, Scrub::PrimaryActive, Scrub::ReservingReplicas, (boost::statechart::history_mode)0>::state(boost::statechart::state<Scrub::Session, Scrub::PrimaryActive, Scrub::ReservingReplicas, (boost::statechart::history_mode)0>::my_context)" when it could be moved instead. 180 : my_base(ctx) 181 , NamedSimply(context<ScrubMachine>().m_scrbr, "PrimaryActive/Session") 182 { 183 dout(10) << "-- state -->> PrimaryActive/Session" << dendl; 184 DECLARE_LOCALS; // 'scrbr' & 'pg_id' aliases 185 ** CID 1585266: Memory - illegal accesses (WRAPPER_ESCAPE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_gc.cc: 231 in RGWGC::remove(int, const std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>> &, librados::v14_2_0::AioCompletion **, optional_yield)() ________________________________________________________________________________________________________ *** CID 1585266: Memory - illegal accesses (WRAPPER_ESCAPE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_gc.cc: 231 in RGWGC::remove(int, const std::vector<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::allocator<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>> &, librados::v14_2_0::AioCompletion **, optional_yield)() 225 ObjectWriteOperation op; 226 cls_rgw_gc_remove(op, tags); 227 228 aio_completion_ptr c{librados::Rados::aio_create_completion(nullptr, nullptr)}; 229 int ret = store->gc_aio_operate(obj_names[index], c.get(), &op); 230 if (ret >= 0) { destroyed when it exits scope. 231 *pc = c.get(); 232 c.release(); 233 } 234 return ret; 235 } 236 ** CID 1585265: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_data_sync.cc: 1940 in RGWDataIncSyncShardCR::RGWDataIncSyncShardCR(RGWDataSyncCtx *, const rgw_pool &, unsigned int, rgw_data_sync_marker &, std::shared_ptr<RGWSyncTraceNode>, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const rgw_raw_obj &, boost::intrusive_ptr<const RGWContinuousLeaseCR>, const rgw_data_sync_status &, RGWObjVersionTracker &, const boost::intrusive_ptr<rgw::bucket_sync::Cache> &, std::mutex &, boost::container::flat_set<rgw_data_notify_entry, std::less<rgw_data_notify_entry>, void> &)() ________________________________________________________________________________________________________ *** CID 1585265: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/rados/rgw_data_sync.cc: 1940 in RGWDataIncSyncShardCR::RGWDataIncSyncShardCR(RGWDataSyncCtx *, const rgw_pool &, unsigned int, rgw_data_sync_marker &, std::shared_ptr<RGWSyncTraceNode>, const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, const rgw_raw_obj &, boost::intrusive_ptr<const RGWContinuousLeaseCR>, const rgw_data_sync_status &, RGWObjVersionTracker &, const boost::intrusive_ptr<rgw::bucket_sync::Cache> &, std::mutex &, boost::container::flat_set<rgw_data_notify_entry, std::less<rgw_data_notify_entry>, void> &)() 1934 const string& status_oid, const rgw_raw_obj& error_repo, 1935 boost::intrusive_ptr<const RGWContinuousLeaseCR> lease_cr, 1936 const rgw_data_sync_status& sync_status, RGWObjVersionTracker& objv, 1937 const boost::intrusive_ptr<rgw::bucket_sync::Cache>& bucket_shard_cache, 1938 ceph::mutex& inc_lock, 1939 bc::flat_set<rgw_data_notify_entry>& modified_shards) "RGWDataBaseSyncShardCR::RGWDataBaseSyncShardCR(RGWDataSyncCtx *, rgw_pool const &, uint32_t, rgw_data_sync_marker &, RGWSyncTraceNodeRef, std::string const &, rgw_raw_obj const &, boost::intrusive_ptr<RGWContinuousLeaseCR const>, rgw_data_sync_status const &, RGWObjVersionTracker &, boost::intrusive_ptr<rgw::bucket_sync::Cache> const &)" when it could be moved instead. 1940 : RGWDataBaseSyncShardCR(sc, pool, shard_id, sync_marker, tn, 1941 status_oid, error_repo, std::move(lease_cr), 1942 sync_status, objv, bucket_shard_cache), 1943 inc_lock(inc_lock), modified_shards(modified_shards) {} 1944 1945 int operate(const DoutPrefixProvider *dpp) override { ** CID 1585264: Uninitialized variables (MISSING_RETURN) /src/static/src/github/ceph/src/test/neorados/read_operations.cc: 268 in NeoRadosReadOps_Checksum_Test::CoTestBody()() ________________________________________________________________________________________________________ *** CID 1585264: Uninitialized variables (MISSING_RETURN) /src/static/src/github/ceph/src/test/neorados/read_operations.cc: 268 in NeoRadosReadOps_Checksum_Test::CoTestBody()() 262 co_await execute(oid, ReadOp{}.checksum(hash_alg::crc32c, seed, 0, 263 data.length(), 4, &crc)); 264 EXPECT_EQ(crc32c(seed, data.substr(0, 4)), crc[0]); 265 EXPECT_EQ(crc32c(seed, data.substr(4, 4)), crc[1]); 266 } 267 268 co_return; 269 } 270 271 CORO_TEST_F(NeoRadosReadOps, RWOrderedRead, ReadOpTest) { 272 buffer::list bl; 273 sys::error_code ec; ** CID 1585263: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /boost/include/boost/json/impl/parser.ipp: 56 in boost::json::parser::reset(boost::json::storage_ptr)() ________________________________________________________________________________________________________ *** CID 1585263: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /boost/include/boost/json/impl/parser.ipp: 56 in boost::json::parser::reset(boost::json::storage_ptr)() 50 51 void 52 parser:: 53 reset(storage_ptr sp) noexcept 54 { 55 p_.reset(); moved instead. 56 p_.handler().st.reset(sp); 57 } 58 59 std::size_t 60 parser:: 61 write_some( ** CID 1585262: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /boost/include/boost/intrusive/hashtable.hpp: 1172 in boost::intrusive::bucket_plus_vtraits<boost::intrusive::bhtraits<LRUSet<LogEntryKey, (int)128>::Node, boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)1, boost::intrusive::dft_tag, (unsigned int)4>, boost::intrusive::bucket_traits_impl<boost::intrusive::bucket_impl<boost::intrusive::slist_node_traits<void *>> *, unsigned long>, (bool)0>::priv_erase_from_single_bucket<boost::intrusive::node_cast_adaptor<boost::intrusive::detail::node_disposer<boost::intrusive::detail::null_disposer, boost::intrusive::bhtraits<LRUSet<LogEntryKey, (int)128>::Node, boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)1, boost::intrusive::dft_tag, (unsigned int)4>, (boost::intrusive::algo_types)3>, boost::intrusive::slist_node<void *> *, boost::intrusive::slist_node<void *> *>>(boost::intrusive::bucket_impl<boost::intrusive::slist_node_traits<void *>> &, boost::intrusive::slist_iterator<boost::intrusive::trivial_value_traits<boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)0>, (bool)0>, boost::intrusive::slist_iterator<boost::intrusive::trivial_value_traits<boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)0>, (bool)0>, T1, boost::move_detail::bool_<(bool)0>)() ________________________________________________________________________________________________________ *** CID 1585262: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /boost/include/boost/intrusive/hashtable.hpp: 1172 in boost::intrusive::bucket_plus_vtraits<boost::intrusive::bhtraits<LRUSet<LogEntryKey, (int)128>::Node, boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)1, boost::intrusive::dft_tag, (unsigned int)4>, boost::intrusive::bucket_traits_impl<boost::intrusive::bucket_impl<boost::intrusive::slist_node_traits<void *>> *, unsigned long>, (bool)0>::priv_erase_from_single_bucket<boost::intrusive::node_cast_adaptor<boost::intrusive::detail::node_disposer<boost::intrusive::detail::null_disposer, boost::intrusive::bhtraits<LRUSet<LogEntryKey, (int)128>::Node, boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)1, boost::intrusive::dft_tag, (unsigned int)4>, (boost::intrusive::algo_types)3>, boost::intrusive::slist_node<void *> *, boost::intrusive::slist_node<void *> *>>(boost::intrusive::bucket_impl<boost::intrusive::slist_node_traits<void *>> &, boost::intrusive::slist_iterator<boost::intrusive::trivial_value_traits<boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)0>, (bool)0>, boost::intrusive::slist_iterator<boost::intrusive::trivial_value_traits<boost::intrusive::slist_node_traits<void *>, (boost::intrusive::link_mode_type)0>, (bool)0>, T1, boost::move_detail::bool_<(bool)0>)() 1166 } 1167 1168 template<class NodeDisposer> 1169 static std::size_t priv_erase_from_single_bucket 1170 (bucket_type &, siterator sbefore_first, siterator slast, NodeDisposer node_disposer, detail::false_) //optimize multikey 1171 { "unlink_after_and_dispose" when it could be moved instead. 1172 return slist_node_algorithms::unlink_after_and_dispose(sbefore_first.pointed_node(), slast.pointed_node(), node_disposer); 1173 } 1174 1175 template<class NodeDisposer> 1176 static void priv_erase_node(bucket_type &b, siterator i, NodeDisposer node_disposer, detail::true_) //optimize multikey 1177 { ** CID 1585261: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/rgw_sal_filter.cc: 722 in rgw::sal::FilterBucket::get_multipart_upload(const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, std::optional<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>, ACLOwner, std::chrono::time_point<ceph::real_clock, std::chrono::duration<unsigned long, std::ratio<(long)1, (long)1000000000>>>)() ________________________________________________________________________________________________________ *** CID 1585261: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/rgw_sal_filter.cc: 722 in rgw::sal::FilterBucket::get_multipart_upload(const std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>> &, std::optional<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>>, ACLOwner, std::chrono::time_point<ceph::real_clock, std::chrono::duration<unsigned long, std::ratio<(long)1, (long)1000000000>>>)() 716 std::unique_ptr<MultipartUpload> FilterBucket::get_multipart_upload( 717 const std::string& oid, 718 std::optional<std::string> upload_id, 719 ACLOwner owner, ceph::real_time mtime) 720 { 721 std::unique_ptr<MultipartUpload> nmu = "get_multipart_upload" when it could be moved instead. 722 next->get_multipart_upload(oid, upload_id, owner, mtime); 723 724 return std::make_unique<FilterMultipartUpload>(std::move(nmu), this); 725 } 726 727 int FilterBucket::list_multiparts(const DoutPrefixProvider *dpp, ** CID 1585260: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/dbstore/sqlite/sqliteDB.h: 355 in SQLPutObjectData::SQLPutObjectData(sqlite3 **, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, ceph::common::CephContext *)() ________________________________________________________________________________________________________ *** CID 1585260: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /src/static/src/github/ceph/src/rgw/driver/dbstore/sqlite/sqliteDB.h: 355 in SQLPutObjectData::SQLPutObjectData(sqlite3 **, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, ceph::common::CephContext *)() 349 private: 350 sqlite3 **sdb = NULL; 351 sqlite3_stmt *stmt = NULL; // Prepared statement 352 353 public: 354 SQLPutObjectData(void **db, std::string db_name, CephContext *cct) : SQLiteDB((sqlite3 *)(*db), db_name, cct), sdb((sqlite3 **)db) {} "SQLiteDB::SQLiteDB(sqlite3 *, std::string, ceph::common::CephContext *)" when it could be moved instead. 355 SQLPutObjectData(sqlite3 **sdbi, std::string db_name, CephContext *cct) : SQLiteDB(*sdbi, db_name, cct), sdb(sdbi) {} 356 357 ~SQLPutObjectData() { 358 if (stmt) 359 sqlite3_finalize(stmt); 360 } ** CID 1585259: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /usr/include/c++/12/bits/stl_algobase.h: 1555 in std::equal<ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>, ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>>(T1, T1, T2)() ________________________________________________________________________________________________________ *** CID 1585259: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) /usr/include/c++/12/bits/stl_algobase.h: 1555 in std::equal<ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>, ceph::buffer::v15_2_0::list::iterator_impl<(bool)1>>(T1, T1, T2)() 1549 __glibcxx_function_requires(_InputIteratorConcept<_II2>) 1550 __glibcxx_function_requires(_EqualOpConcept< 1551 typename iterator_traits<_II1>::value_type, 1552 typename iterator_traits<_II2>::value_type>) 1553 __glibcxx_requires_can_increment_range(__first1, __last1, __first2); 1554 could be moved instead. 1555 return std::__equal_aux(__first1, __last1, __first2); 1556 } 1557 1558 /** 1559 * @brief Tests a range for element-wise equality. 1560 * @ingroup non_mutating_algorithms ________________________________________________________________________________________________________ To view the defects in Coverity Scan visit, https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P… To manage Coverity Scan email notifications for &quot;kkeithle(a)redhat.com&quot;.com", click https://u15810271.ct.sendgrid.net/ls/click?upn=HRESupC-2F2Czv4BOaCWWCy7my0P… -- Kaleb