From af7870f0a24b7bcf073db8ccfa0eb811b8fc5d41 Mon Sep 17 00:00:00 2001 From: galsalomon66 Date: Mon, 27 Oct 2025 17:25:58 +0000 Subject: [PATCH] initializing of enable_progress length_before_processing length_post_processing on construction. these variable are getting initialized on s3select/CSV flow, no valgrind local run had discovered any issue related to these variables. valgrind reports produced by teuthology points on run_s3select_on_csv to contain UninitCondition warning. sometimes. Signed-off-by: galsalomon66 --- src/rgw/rgw_s3select.cc | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/rgw/rgw_s3select.cc b/src/rgw/rgw_s3select.cc index c001070eb8153..f37f53cdd7551 100644 --- a/src/rgw/rgw_s3select.cc +++ b/src/rgw/rgw_s3select.cc @@ -285,6 +285,7 @@ RGWSelectObj_ObjStore_S3::RGWSelectObj_ObjStore_S3(): m_start_scan_sz(0), m_end_scan_sz(0), m_object_size_for_processing(0), + enable_progress(false), m_parquet_type(false), m_json_type(false), chunk_number(0), @@ -388,7 +389,7 @@ int RGWSelectObj_ObjStore_S3::get_params(optional_yield y) int RGWSelectObj_ObjStore_S3::run_s3select_on_csv(const char* query, const char* input, size_t input_length) { int status = 0; - uint32_t length_before_processing, length_post_processing; + uint32_t length_before_processing = 0, length_post_processing = 0; csv_object::csv_defintions csv; s3select_syntax.parse_query(query); -- 2.39.5