|
 |
0f0231 |
From f1e404c33ad14f6784aa1926ae75fa49e614748e Mon Sep 17 00:00:00 2001
|
|
|
0f0231 |
From: Ming-Hung Tsai <mtsai@redhat.com>
|
|
|
0f0231 |
Date: Wed, 2 Jun 2021 12:14:34 +0800
|
|
|
0f0231 |
Subject: [PATCH 07/10] [all] Fix uninitialized class members
|
|
|
0f0231 |
|
|
|
0f0231 |
---
|
|
|
0f0231 |
era/restore_emitter.cc | 1 +
|
|
|
0f0231 |
thin-provisioning/metadata_dumper.cc | 10 +++++++---
|
|
|
0f0231 |
thin-provisioning/thin_delta.cc | 4 +++-
|
|
|
0f0231 |
3 files changed, 11 insertions(+), 4 deletions(-)
|
|
|
0f0231 |
|
|
|
0f0231 |
diff --git a/era/restore_emitter.cc b/era/restore_emitter.cc
|
|
|
0f0231 |
index c09fa66..7fbac7d 100644
|
|
|
0f0231 |
--- a/era/restore_emitter.cc
|
|
|
0f0231 |
+++ b/era/restore_emitter.cc
|
|
|
0f0231 |
@@ -14,6 +14,7 @@ namespace {
|
|
|
0f0231 |
: md_(md),
|
|
|
0f0231 |
in_superblock_(false),
|
|
|
0f0231 |
in_writeset_(false),
|
|
|
0f0231 |
+ era_(0),
|
|
|
0f0231 |
in_era_array_(false) {
|
|
|
0f0231 |
}
|
|
|
0f0231 |
|
|
|
0f0231 |
diff --git a/thin-provisioning/metadata_dumper.cc b/thin-provisioning/metadata_dumper.cc
|
|
|
0f0231 |
index 4feb40f..665c762 100644
|
|
|
0f0231 |
--- a/thin-provisioning/metadata_dumper.cc
|
|
|
0f0231 |
+++ b/thin-provisioning/metadata_dumper.cc
|
|
|
0f0231 |
@@ -180,7 +180,7 @@ namespace {
|
|
|
0f0231 |
// This is about classifying and summarising btree nodes. The use of a btree
|
|
|
0f0231 |
// node may not be obvious when inspecting it in isolation. But more information
|
|
|
0f0231 |
// may be gleaned by examining child and sibling nodes.
|
|
|
0f0231 |
-//
|
|
|
0f0231 |
+//
|
|
|
0f0231 |
// So the process is:
|
|
|
0f0231 |
// - scan every metadata block, summarising it's potential uses.
|
|
|
0f0231 |
// - repeatedly iterate those summaries until we can glean no more useful information.
|
|
|
0f0231 |
@@ -474,7 +474,7 @@ namespace {
|
|
|
0f0231 |
node_info get_internal_info(block_manager::read_ref &rr) {
|
|
|
0f0231 |
node_info info;
|
|
|
0f0231 |
info.b = rr.get_location();
|
|
|
0f0231 |
-
|
|
|
0f0231 |
+
|
|
|
0f0231 |
// values refer to blocks, so we should have infos for them.
|
|
|
0f0231 |
auto n = to_node<block_traits>(rr);
|
|
|
0f0231 |
::uint64_t key_low = 0;
|
|
|
0f0231 |
@@ -524,7 +524,7 @@ namespace {
|
|
|
0f0231 |
node_info info;
|
|
|
0f0231 |
info.b = rr.get_location();
|
|
|
0f0231 |
|
|
|
0f0231 |
- auto vsize = to_cpu<uint32_t>(hdr.value_size);
|
|
|
0f0231 |
+ auto vsize = to_cpu<uint32_t>(hdr.value_size);
|
|
|
0f0231 |
info.values = to_cpu<uint32_t>(hdr.nr_entries);
|
|
|
0f0231 |
|
|
|
0f0231 |
if (vsize == sizeof(device_details_traits::disk_type)) {
|
|
|
0f0231 |
@@ -645,6 +645,10 @@ namespace {
|
|
|
0f0231 |
public:
|
|
|
0f0231 |
mapping_emit_visitor(emitter::ptr e)
|
|
|
0f0231 |
: e_(e),
|
|
|
0f0231 |
+ origin_start_(0),
|
|
|
0f0231 |
+ dest_start_(0),
|
|
|
0f0231 |
+ time_(0),
|
|
|
0f0231 |
+ len_(0),
|
|
|
0f0231 |
in_range_(false) {
|
|
|
0f0231 |
}
|
|
|
0f0231 |
|
|
|
0f0231 |
diff --git a/thin-provisioning/thin_delta.cc b/thin-provisioning/thin_delta.cc
|
|
|
0f0231 |
index 5b31608..ee48dea 100644
|
|
|
0f0231 |
--- a/thin-provisioning/thin_delta.cc
|
|
|
0f0231 |
+++ b/thin-provisioning/thin_delta.cc
|
|
|
0f0231 |
@@ -259,7 +259,9 @@ namespace local {
|
|
|
0f0231 |
class simple_emitter : public diff_emitter {
|
|
|
0f0231 |
public:
|
|
|
0f0231 |
simple_emitter(indented_stream &out)
|
|
|
0f0231 |
- : diff_emitter(out) {
|
|
|
0f0231 |
+ : diff_emitter(out),
|
|
|
0f0231 |
+ vbegin_(0),
|
|
|
0f0231 |
+ vend_(0) {
|
|
|
0f0231 |
}
|
|
|
0f0231 |
|
|
|
0f0231 |
void left_only(uint64_t vbegin, uint64_t dbegin, uint64_t len) {
|
|
|
0f0231 |
--
|
|
|
0f0231 |
1.8.3.1
|
|
|
0f0231 |
|