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

lbmpdm: fix issue found by PVS Studio (V519) 

The 'msgid.msg_def_id' variable is assigned values twice successively.
The 'msgid.ver_major' variable is assigned values twice successively.
The 'msgid.ver_minor' variable is assigned values twice successively.

Change-Id: I675d5a0870575ac358173c12920e0c0c920f17b1
Reviewed-on: https://code.wireshark.org/review/27595
Reviewed-by: Anders Broman <a.broman58@gmail.com>
This commit is contained in:
Alexis La Goutte 2018-05-16 08:32:31 +02:00 committed by Anders Broman
parent 6a95d5251c
commit 57ddb2e651
1 changed files with 0 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
epan/dissectors/packet-lbmpdm.c
View File

@ -1202,9 +1202,6 @@ int lbmpdm_dissect_lbmpdm_payload(tvbuff_t * tvb, int offset, packet_info * pinf
msglen = (int)raw_msglen;
msgid.channel = channel;
msgid.msg_def_id = 0;
msgid.ver_major = 0;
msgid.ver_minor = 0;
msgid.offset_table = NULL;
subtree_item = proto_tree_add_protocol_format(tree, proto_lbmpdm, tvb, offset, msglen, "LBMPDM Protocol");
subtree = proto_item_add_subtree(subtree_item, ett_lbmpdm);