diff options
author | Ben Chan <benchan@chromium.org> | 2013-09-04 15:31:34 -0700 |
---|---|---|
committer | Aleksander Morgado <aleksander@lanedo.com> | 2013-09-05 07:35:03 +0200 |
commit | eb54d22b247567ba908aadaab7710d6773c80c64 (patch) | |
tree | 55682bffb82ede73ece35892d83e8559f2015d9c | |
parent | d9c4a07d999bbbba70bf5e904737fdb2911eece1 (diff) |
huawei: fix uninitialized variable issues
This patch fixes the following uninitialized variable issues, which was
introduced in the previous commit "huawei: retry connect/disconnect attempt
upon ^NDISSTATQRY? failures" (commit 57c657bd066366db6892ac2a8adfec2ca209ccbe).
huawei/mm-broadband-bearer-huawei.c:127:9: error: variable 'ipv4_available' is used uninitialized whenever '||' condition is true
[-Werror,-Wsometimes-uninitialized]
if (!response ||
^~~~~~~~~
huawei/mm-broadband-bearer-huawei.c:141:9: note: uninitialized use occurs here
if (ipv4_available && ipv4_connected) {
^~~~~~~~~~~~~~
huawei/mm-broadband-bearer-huawei.c:127:9: note: remove the '||' if its condition is always false
if (!response ||
^~~~~~~~~~~~
huawei/mm-broadband-bearer-huawei.c:115:28: note: initialize the variable 'ipv4_available' to silence this warning
gboolean ipv4_available;
^
= 0
huawei/mm-broadband-bearer-huawei.c:484:9: error: variable 'ipv4_available' is used uninitialized whenever '||' condition is true
[-Werror,-Wsometimes-uninitialized]
if (!response ||
^~~~~~~~~
huawei/mm-broadband-bearer-huawei.c:498:9: note: uninitialized use occurs here
if (ipv4_available && !ipv4_connected) {
^~~~~~~~~~~~~~
huawei/mm-broadband-bearer-huawei.c:484:9: note: remove the '||' if its condition is always false
if (!response ||
^~~~~~~~~~~~
huawei/mm-broadband-bearer-huawei.c:472:28: note: initialize the variable 'ipv4_available' to silence this warning
gboolean ipv4_available;
^
= 0
-rw-r--r-- | plugins/huawei/mm-broadband-bearer-huawei.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/plugins/huawei/mm-broadband-bearer-huawei.c b/plugins/huawei/mm-broadband-bearer-huawei.c index 38892d94..eec37f3d 100644 --- a/plugins/huawei/mm-broadband-bearer-huawei.c +++ b/plugins/huawei/mm-broadband-bearer-huawei.c @@ -112,10 +112,10 @@ connect_ndisstatqry_check_ready (MMBaseModem *modem, Connect3gppContext *ctx; const gchar *response; GError *error = NULL; - gboolean ipv4_available; - gboolean ipv4_connected; - gboolean ipv6_available; - gboolean ipv6_connected; + gboolean ipv4_available = FALSE; + gboolean ipv4_connected = FALSE; + gboolean ipv6_available = FALSE; + gboolean ipv6_connected = FALSE; ctx = self->priv->connect_pending; g_assert (ctx != NULL); @@ -469,10 +469,10 @@ disconnect_ndisstatqry_check_ready (MMBaseModem *modem, Disconnect3gppContext *ctx; const gchar *response; GError *error = NULL; - gboolean ipv4_available; - gboolean ipv4_connected; - gboolean ipv6_available; - gboolean ipv6_connected; + gboolean ipv4_available = FALSE; + gboolean ipv4_connected = FALSE; + gboolean ipv6_available = FALSE; + gboolean ipv6_connected = FALSE; ctx = self->priv->disconnect_pending; g_assert (ctx != NULL); |