From 96c0700c62721e89fe1496285e4ef6454a7800d5 Mon Sep 17 00:00:00 2001 From: Arkady Gilinsky <8351139-ark-g@users.noreply.gitlab.com> Date: Thu, 15 Jul 2021 07:26:11 +0300 Subject: [PATCH] wireshark(HEAD): oampdu: Fix: do not stop GetReq packet parsing for Object 0 * The next_byte variable is taken before the pointer moved forward, this lead to stop parsing get request packets when object is 0. This commit fixes it. Signed-off-by: Arkady Gilinsky <8351139-ark-g@users.noreply.gitlab.com> --- epan/dissectors/packet-oampdu.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/epan/dissectors/packet-oampdu.c b/epan/dissectors/packet-oampdu.c index 9292a2a0a8..11a69f3542 100644 --- a/epan/dissectors/packet-oampdu.c +++ b/epan/dissectors/packet-oampdu.c @@ -1711,8 +1711,8 @@ dissect_oampdu_vendor_specific(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tr } } } - next_byte = tvb_get_guint8(tvb, offset); offset += variable_length; + next_byte = tvb_get_guint8(tvb, offset); while (next_byte != 0x00) { proto_tree_add_item(dpoe_opcode_tree, hf_dpoe_variable_descriptor, tvb, offset, 3, ENC_BIG_ENDIAN); offset += 3;