Merge pull request #16867 from amitkumar50/cov-librbd-1

librbd: Initializing member variables Reviewed-by: Jason Dillaman <dillaman@redhat.com>
ceph · Aug 7, 2017 · 52e8f3b · 52e8f3b
2 parents 5fbd8dc + 2e887e7
commit 52e8f3b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/librbd/WatchNotifyTypes.h b/src/librbd/WatchNotifyTypes.h
@@ -228,7 +228,7 @@ struct SnapRenamePayload : public SnapPayloadBase {
 		    const std::string &dst_name)
     : SnapPayloadBase(cls::rbd::UserSnapshotNamespace(), dst_name), snap_id(src_snap_id) {}
 
-  uint64_t snap_id;
+  uint64_t snap_id = 0;
 
   void encode(bufferlist &bl) const;
   void decode(__u8 version, bufferlist::iterator &iter);

diff --git a/src/librbd/operation/FlattenRequest.h b/src/librbd/operation/FlattenRequest.h
@@ -73,7 +73,7 @@ class FlattenRequest : public Request<ImageCtxT>
   uint64_t m_overlap_objects;
   ::SnapContext m_snapc;
   ProgressContext &m_prog_ctx;
-  State m_state;
+  State m_state = STATE_FLATTEN_OBJECTS;
 
   ParentSpec m_parent_spec;
   bool m_ignore_enoent;

diff --git a/src/librbd/operation/RebuildObjectMapRequest.h b/src/librbd/operation/RebuildObjectMapRequest.h
@@ -63,7 +63,7 @@ class RebuildObjectMapRequest : public AsyncRequest<ImageCtxT> {
 
   ImageCtxT &m_image_ctx;
   ProgressContext &m_prog_ctx;
-  State m_state;
+  State m_state = STATE_RESIZE_OBJECT_MAP;
   bool m_attempted_trim;
 
   void send_resize_object_map();