From: Kefu Chai Date: Mon, 12 Apr 2021 09:32:59 +0000 (+0800) Subject: mon,osd: always init local variable X-Git-Tag: v17.1.0~2261^2~1 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=7c2f58ea5ec70e5606ab362e6f4cfda087cec65a;p=ceph.git mon,osd: always init local variable 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 --- diff --git a/src/mon/Monitor.cc b/src/mon/Monitor.cc index 6126db9caa60..ffac8be278c1 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 c0fe8f781c21..fe2fe7defa9a 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; }