aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorErwin Rol <erwin@erwinrol.com>2017-11-25 17:34:24 +0100
committerMichael Mann <mmann78@netscape.net>2017-11-26 03:09:21 +0000
commit076fee6e636db2123744c1ecf004e9c6ff9b0a8c (patch)
tree2c3796188cf2d02e8faf6edd861f13e9f696391a
parent18657caf8d5c5388190b29c0cc1b4b94c997d5dc (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.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/epan/dissectors/packet-artnet.c b/epan/dissectors/packet-artnet.c
index a1a633c..8e73368 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;