osmith
/
wireshark
forked from osmocom/wireshark
1
0
Fork 0
Code Pull Requests Releases Wiki Activity

s101: add initializer. 

../epan/dissectors/packet-s101.c: In function ‘dissect_S101’:
../epan/dissectors/packet-s101.c:279:53: error: ‘app_bytes_len’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
          ((*flags & 0xF) || (S101_DTD_GLOW != *dtd) || (APP_BYTES_LEN != app_bytes_len)))) {
                                                     ^
../epan/dissectors/packet-s101.c:241:12: note: ‘app_bytes_len’ was declared here
     guint8 app_bytes_len;
            ^
cc1: all warnings being treated as errors

Change-Id: Ibc7ab9206aeda2afe895a1813e7d4cb196b41817
Reviewed-on: https://code.wireshark.org/review/28105
Petri-Dish: Dario Lombardo <lomato@gmail.com>
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
This commit is contained in:
Dario Lombardo 2018-06-08 08:57:02 +02:00 committed by Pascal Quantin
parent 1c7b101b35
commit 7ae1c683f3
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
epan/dissectors/packet-s101.c
View File

@ -238,7 +238,7 @@ static int
find_s101_packet_header(tvbuff_t *tvb, int* offset, guint8 *start, guint8 *slot, guint8 *message, guint8 *version, guint8 *dtd, guint8 *command,
guint8 *flags, guint8* app_bytes, guint64 *msgLength, guint16 *crc)
{
guint8 app_bytes_len;
guint8 app_bytes_len = 0;
int i;
*start = tvb_get_guint8(tvb, *offset); // no CRC and no escaping on first bytes.