base: Fix unitialized storage
authorDaniel R. Carvalho <odanrc@yahoo.com.br>
Tue, 15 Jan 2019 10:51:46 +0000 (11:51 +0100)
committerDaniel Carvalho <odanrc@yahoo.com.br>
Fri, 18 Jan 2019 12:46:31 +0000 (12:46 +0000)
The bitunion is not being initialized on constructor to avoid
performance overhead, and that generated a maybe-unitialized
error when a sub-class was being copied before assigned in
serialize's parseParam() in some compilers.

This patch adds zero-initialization to the problematic variable
to appease the compiler.

Change-Id: I90fa6aa356b3e14ec25e3294b17ed10f429a9a38
Signed-off-by: Daniel R. Carvalho <odanrc@yahoo.com.br>
Reviewed-on: https://gem5-review.googlesource.com/c/15635
Reviewed-by: Gabe Black <gabeblack@google.com>
Maintainer: Gabe Black <gabeblack@google.com>

src/sim/serialize.hh

index 543477726c83f13e72cea47e02d4add2e776b296..a45d1bbcdd3304559b7651b2041e13bae3dfb459 100644 (file)
@@ -289,7 +289,8 @@ template <class T>
 bool
 parseParam(const std::string &s, BitUnionType<T> &value)
 {
-    auto storage = static_cast<BitUnionBaseType<T>>(value);
+    // Zero initialize storage to avoid leaking an uninitialized value
+    BitUnionBaseType<T> storage = BitUnionBaseType<T>();
     auto res = to_number(s, storage);
     value = storage;
     return res;