diff options
author | Jaap Keuter <jaap.keuter@xs4all.nl> | 2010-07-27 06:46:00 +0000 |
---|---|---|
committer | Jaap Keuter <jaap.keuter@xs4all.nl> | 2010-07-27 06:46:00 +0000 |
commit | ccf61ec3b2cb94df33a1b77983767d8fef7cf7cb (patch) | |
tree | a6bbfb0ff8b90910804de6ee7fb8ff077c114fce /plugins/unistim | |
parent | f0bb5f4c9aa425f3b08e0d96139db0910fadd134 (diff) |
From Don Newton:
Fails to decrement msg_len on 3 different message types which causes the
dissector to attempt to decode an addition or misplaced unistim cmd.
svn path=/trunk/; revision=33643
Diffstat (limited to 'plugins/unistim')
-rw-r--r-- | plugins/unistim/packet-unistim.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/plugins/unistim/packet-unistim.c b/plugins/unistim/packet-unistim.c index d692bd820a..e8491ed2cd 100644 --- a/plugins/unistim/packet-unistim.c +++ b/plugins/unistim/packet-unistim.c @@ -643,7 +643,7 @@ dissect_broadcast_switch(proto_tree *msg_tree, proto_tree *time_tree; bcast_cmd=tvb_get_guint8(tvb,offset); proto_tree_add_item(msg_tree,hf_broadcast_switch_cmd,tvb,offset,1,FALSE); - offset+=1; + offset+=1;msg_len-=1; switch(bcast_cmd){ case 0x00: /*Accessory Sync Update - len=3 */ @@ -1353,7 +1353,7 @@ dissect_display_phone(proto_tree *msg_tree, guint highlight_cmd; display_cmd=tvb_get_guint8(tvb,offset); proto_tree_add_item(msg_tree,hf_display_phone_cmd,tvb,offset,1,FALSE); - offset+=1; + offset+=1;msg_len-=1; switch(display_cmd){ case 0x00: /*Display Manager Attributes Info*/ @@ -1803,7 +1803,7 @@ dissect_network_phone(proto_tree *msg_tree, guint i; network_cmd=tvb_get_guint8(tvb,offset); proto_tree_add_item(msg_tree,hf_network_phone_cmd,tvb,offset,1,FALSE); - offset+=1; + offset+=1;msg_len-=1; switch(network_cmd){ case 0x00: /*Soft Reset Ack done length = 3*/ |