]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
messages: Initialization of uninitialized members varied classes 16848/head
authoramitkuma <amitkuma@redhat.com>
Sun, 6 Aug 2017 17:41:10 +0000 (23:11 +0530)
committeramitkuma <amitkuma@redhat.com>
Sun, 6 Aug 2017 17:41:10 +0000 (23:11 +0530)
Fixes the coverity issues:

** 1188168 Uninitialized scalar field
CID 1188168 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
2. uninit_member: Non-static class member success is not initialized
in this constructor nor in any functions that it calls.

** 1188169 Uninitialized scalar field
2. uninit_member: Non-static class member map_epoch is not initialized
in this constructor nor in any functions that it calls.
CID 1188169 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member min_epoch is not initialized
in this constructor nor in any functions that it calls.

** 1188170 Uninitialized scalar field
2. uninit_member: Non-static class member map_epoch is not initialized
in this constructor nor in any functions that it calls.
CID 1188170 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member min_epoch is not initialized
in this constructor nor in any functions that it calls.

** 1188171 Uninitialized scalar field
2. uninit_member: Non-static class member map_epoch is not initialized
in this constructor nor in any functions that it calls.
CID 1188171 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member min_epoch is not initialized
in this constructor nor in any functions that it calls.

** 1188172 Uninitialized scalar field
2. uninit_member: Non-static class member map_epoch is not initialized
 in this constructor nor in any functions that it calls.
CID 1188172 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member min_epoch is not initialized
 in this constructor nor in any functions that it calls.

** 1188173 Uninitialized scalar field
2. uninit_member: Non-static class member map_epoch is not initialized
 in this constructor nor in any functions that it calls.
CID 1188173 (#1 of 1): Uninitialized scalar field (UNINIT_CTOR)
4. uninit_member: Non-static class member min_epoch is not initialized
 in this constructor nor in any functions that it calls.

Signed-off-by: Amit Kumar amitkuma@redhat.com
src/messages/MExportDirPrepAck.h
src/messages/MOSDECSubOpRead.h
src/messages/MOSDECSubOpReadReply.h
src/messages/MOSDECSubOpWrite.h
src/messages/MOSDECSubOpWriteReply.h

index bb9bb13778d533caddac17699cd2dce6654632a5..d623d1acc0962213879d64ea00e4cc9692b31007 100644 (file)
@@ -20,7 +20,7 @@
 
 class MExportDirPrepAck : public Message {
   dirfrag_t dirfrag;
-  bool success;
+  bool success = false;
 
  public:
   dirfrag_t get_dirfrag() { return dirfrag; }
index de9317e45a63c6920d66ecbf783e0b8acd0999e9..48a283c2060149b6a5619991059bccbab20db243 100644 (file)
@@ -24,7 +24,7 @@ class MOSDECSubOpRead : public MOSDFastDispatchOp {
 
 public:
   spg_t pgid;
-  epoch_t map_epoch, min_epoch;
+  epoch_t map_epoch = 0, min_epoch = 0;
   ECSubRead op;
 
   int get_cost() const override {
index 7ad931445c9778270a1256e97a03c0955f12802f..4cc3547400b5b80d0c0227b4a3341c81c6daeca8 100644 (file)
@@ -24,7 +24,7 @@ class MOSDECSubOpReadReply : public MOSDFastDispatchOp {
 
 public:
   spg_t pgid;
-  epoch_t map_epoch, min_epoch;
+  epoch_t map_epoch = 0, min_epoch = 0;
   ECSubReadReply op;
 
   int get_cost() const override {
index 72a272c6ab02a5da889ca583329e9578ece2d31c..b1e6b8c361f922dd295f02c6698f8d62ae666873 100644 (file)
@@ -24,7 +24,7 @@ class MOSDECSubOpWrite : public MOSDFastDispatchOp {
 
 public:
   spg_t pgid;
-  epoch_t map_epoch, min_epoch;
+  epoch_t map_epoch = 0, min_epoch = 0;
   ECSubWrite op;
 
   int get_cost() const override {
index 778a13d798494c970e962aaabacf5b82dc70aaec..e7ca5233bed6c9d203e76e8475701b678e4a996a 100644 (file)
@@ -24,7 +24,7 @@ class MOSDECSubOpWriteReply : public MOSDFastDispatchOp {
 
 public:
   spg_t pgid;
-  epoch_t map_epoch, min_epoch;
+  epoch_t map_epoch = 0, min_epoch = 0;
   ECSubWriteReply op;
 
   int get_cost() const override {