diff options
author | João Valverde <joao.valverde@tecnico.ulisboa.pt> | 2017-09-07 13:02:22 +0100 |
---|---|---|
committer | João Valverde <j@v6e.pt> | 2017-09-07 18:32:15 +0000 |
commit | db63f75eab219c16c450d4fcd57bfeae1bcb8cb6 (patch) | |
tree | 7774b263bf012cce873978ed95c4dc2a90e9d818 /epan/dissectors/packet-opensafety.c | |
parent | d38197bb676d8937f5d9acd534200195b2e541bc (diff) |
More [-Wmaybe-uninitialized] fixes with -Og
There may be some amount of bogosity involved but initialize the
variables and add a default case to prevent the noise and the build
from breaking with -Werror.
Change-Id: I20432ea74a1e5edc28be75a97077c9aa7bc87a35
Reviewed-on: https://code.wireshark.org/review/23426
Petri-Dish: João Valverde <j@v6e.pt>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: João Valverde <j@v6e.pt>
Diffstat (limited to 'epan/dissectors/packet-opensafety.c')
-rw-r--r-- | epan/dissectors/packet-opensafety.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/packet-opensafety.c b/epan/dissectors/packet-opensafety.c index f8d80f4a47..c0e046402d 100644 --- a/epan/dissectors/packet-opensafety.c +++ b/epan/dissectors/packet-opensafety.c @@ -545,10 +545,10 @@ static gboolean findSafetyFrame ( tvbuff_t *message_tvb, guint u_Offset, gboolea guint *u_frameOffset, guint *u_frameLength, opensafety_packet_info *packet ) { guint ctr, rem_length; - guint16 crc, f2crc, calcCrc; - guint8 b_Length, b_CTl, crcOffset, crc1Type; + guint16 crc, f2crc, calcCrc = 0; + guint8 b_Length = 0, b_CTl = 0, crcOffset = 0, crc1Type = 0; guint8 *bytes; - guint b_ID; + guint b_ID = 0; gboolean found; found = FALSE; |