diff options
author | Erwin Rol <erwin@erwinrol.com> | 2017-11-25 17:34:24 +0100 |
---|---|---|
committer | Michael Mann <mmann78@netscape.net> | 2017-11-26 03:09:21 +0000 |
commit | 076fee6e636db2123744c1ecf004e9c6ff9b0a8c (patch) | |
tree | 2c3796188cf2d02e8faf6edd861f13e9f696391a | |
parent | 18657caf8d5c5388190b29c0cc1b4b94c997d5dc (diff) |
artnet: Fix ArtPollReply spare/style field order
In the ArtPollReply the field order of the style and 3 spare bytes
was wrong, according artnet spec 1.4 page 25 the order should be
spare, spare, spare, style.
Change-Id: I5683e5a8e97643a7bb1962178178c175d485098c
Signed-off-by: Erwin Rol <erwin@erwinrol.com>
Reviewed-on: https://code.wireshark.org/review/24584
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Michael Mann <mmann78@netscape.net>
-rw-r--r-- | epan/dissectors/packet-artnet.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/epan/dissectors/packet-artnet.c b/epan/dissectors/packet-artnet.c index a1a633c9b6..8e733682a7 100644 --- a/epan/dissectors/packet-artnet.c +++ b/epan/dissectors/packet-artnet.c @@ -2253,14 +2253,14 @@ dissect_artnet_poll_reply(tvbuff_t *tvb, guint offset, proto_tree *tree) offset, 1, ENC_BIG_ENDIAN); offset += 1; - proto_tree_add_item(tree, hf_artnet_poll_reply_style, tvb, - offset, 1, ENC_BIG_ENDIAN); - offset += 1; - proto_tree_add_item(tree, hf_artnet_spare, tvb, offset, 3, ENC_NA); offset += 3; + proto_tree_add_item(tree, hf_artnet_poll_reply_style, tvb, + offset, 1, ENC_BIG_ENDIAN); + offset += 1; + proto_tree_add_item(tree, hf_artnet_poll_reply_mac, tvb, offset, 6, ENC_NA); offset += 6; |