Avoid a ubsan warning on GCC 4.9 due uninitialized sign enum being

read during swap (in the move constructor)

2 files changed, 1 insertions, 5 deletions

diff --git a/src/lib/math/bigint/bigint.cpp b/src/lib/math/bigint/bigint.cpp
index 45c351256..059b019e4 100644
--- a/src/lib/math/bigint/bigint.cpp
+++ b/src/lib/math/bigint/bigint.cpp
@@ -18,8 +18,6 @@ namespace Botan {
 */
 BigInt::BigInt(u64bit n)
    {
-   set_sign(Positive);
-
    if(n == 0)
       return;
 
@@ -82,7 +80,6 @@ BigInt::BigInt(const std::string& str)
 */
 BigInt::BigInt(const byte input[], size_t length, Base base)
    {
-   set_sign(Positive);
    *this = decode(input, length, base);
    }
 
@@ -91,7 +88,6 @@ BigInt::BigInt(const byte input[], size_t length, Base base)
 */
 BigInt::BigInt(RandomNumberGenerator& rng, size_t bits)
    {
-   set_sign(Positive);
    randomize(rng, bits);
    }
 
diff --git a/src/lib/math/bigint/bigint.h b/src/lib/math/bigint/bigint.h
index 08b7d53fc..a1164bf92 100644
--- a/src/lib/math/bigint/bigint.h
+++ b/src/lib/math/bigint/bigint.h
@@ -518,7 +518,7 @@ class BOTAN_DLL BigInt
 
    private:
       secure_vector<word> m_reg;
-      Sign m_signedness;
+      Sign m_signedness = Positive;
    };
 
 /*