diff options
author | Arkady Gilinsky <8351139-ark-g@users.noreply.gitlab.com> | 2021-07-15 07:26:11 +0300 |
---|---|---|
committer | Wireshark GitLab Utility <gerald+gitlab-utility@wireshark.org> | 2021-07-15 13:01:38 +0000 |
commit | 96c0700c62721e89fe1496285e4ef6454a7800d5 (patch) | |
tree | 8ed0784582ed63a4c22d252f5b81b20fb34e551b | |
parent | 458d870a6686bab176c6f7138ba74a019b1396cc (diff) |
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>
-rw-r--r-- | epan/dissectors/packet-oampdu.c | 2 |
1 files changed, 1 insertions, 1 deletions
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; |