From 7c2f58ea5ec70e5606ab362e6f4cfda087cec65a Mon Sep 17 00:00:00 2001 From: Kefu Chai Date: Mon, 12 Apr 2021 17:32:59 +0800 Subject: [PATCH] mon,osd: always init local variable MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit this change silences the warning like: ../src/mon/Monitor.cc: In member function ‘void Monitor::handle_command(MonOpRequestRef)’: ../src/mon/Monitor.cc:3703:55: warning: ‘osd’ may be used uninitialized in this function [-Wmaybe-uninitialized] 3703 | uint64_t seq = mgrstatmon()->get_last_osd_stat_seq(osd); | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~ Signed-off-by: Kefu Chai --- src/mon/Monitor.cc | 2 +- src/osd/osd_types.h | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc index 6126db9caa609..ffac8be278c16 100644 --- a/src/mon/Monitor.cc +++ b/src/mon/Monitor.cc @@ -3698,7 +3698,7 @@ void Monitor::handle_command(MonOpRequestRef op) rs = ss2.str(); r = 0; } else if (prefix == "osd last-stat-seq") { - int64_t osd; + int64_t osd = 0; cmd_getval(cmdmap, "id", osd); uint64_t seq = mgrstatmon()->get_last_osd_stat_seq(osd); if (f) { diff --git a/src/osd/osd_types.h b/src/osd/osd_types.h index c0fe8f781c21e..fe2fe7defa9a8 100644 --- a/src/osd/osd_types.h +++ b/src/osd/osd_types.h @@ -1631,12 +1631,12 @@ public: } int64_t get_dedup_tier() const { - int64_t tier_id; + int64_t tier_id = 0; opts.get(pool_opts_t::DEDUP_TIER, &tier_id); return tier_id; } int64_t get_dedup_cdc_chunk_size() const { - int64_t chunk_size; + int64_t chunk_size = 0; opts.get(pool_opts_t::DEDUP_CDC_CHUNK_SIZE, &chunk_size); return chunk_size; } -- 2.39.5