From 03ddd5b769b92d2c1202efc42a1abfd3a68b5f05 Mon Sep 17 00:00:00 2001 From: Matty Williams Date: Thu, 19 Feb 2026 15:07:07 +0000 Subject: [PATCH] cls/test: Make function parameters by reference not by value Stops the cls_fifo.get_info test from Segmentation Faulting. Fixes: https://tracker.ceph.com/issues/74648 Signed-off-by: Matty Williams --- src/neorados/cls/fifo.h | 2 +- src/test/cls_fifo/ceph_test_neocls_fifo.cc | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/neorados/cls/fifo.h b/src/neorados/cls/fifo.h index 484dcd44d393..3f15a834bf66 100644 --- a/src/neorados/cls/fifo.h +++ b/src/neorados/cls/fifo.h @@ -361,7 +361,7 @@ public: template CompletionToken> - static auto get_meta(neorados::RADOS rados, Object obj, IOContext ioc, + static auto get_meta(neorados::RADOS& rados, Object obj, IOContext ioc, std::optional objv, CompletionToken&& token) { diff --git a/src/test/cls_fifo/ceph_test_neocls_fifo.cc b/src/test/cls_fifo/ceph_test_neocls_fifo.cc index 7b2020fdf1d0..fd3017007796 100644 --- a/src/test/cls_fifo/ceph_test_neocls_fifo.cc +++ b/src/test/cls_fifo/ceph_test_neocls_fifo.cc @@ -46,13 +46,13 @@ namespace fifo = rados::cls::fifo; class FIFOtest { public: template - static auto create_meta(neorados::RADOS rados, Args&&... args) { + static auto create_meta(neorados::RADOS& rados, Args&&... args) { return detail::FIFOImpl::create_meta(rados, std::forward(args)...); } template - static auto get_meta(neorados::RADOS rados, Args&&... args) { + static auto get_meta(neorados::RADOS& rados, Args&&... args) { return FIFO::get_meta(rados, std::forward(args)...); } -- 2.47.3