diff options
author | Guy Harris <guy@alum.mit.edu> | 2019-05-12 11:20:30 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2019-05-12 18:21:09 +0000 |
commit | beed21b038683377efb7b5c13b36a3a35fa720a2 (patch) | |
tree | c18886aa0252aa20d9357e62e315d20f8d30a929 /epan/dissectors/packet-isis-clv.c | |
parent | c135fb1308e5f467722ea7861114e0d0f445f947 (diff) |
Fix the length of the item added for the value field of undissected CLVs.
The length variable is the length of the value, not the length of the
entire CLV, so there's no need to subtract the length of the C and the
L from the length - it covers just the V.
Change-Id: I711657e4e0b76e2aac9d58efd88f45201b9c2c5b
Reviewed-on: https://code.wireshark.org/review/33174
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'epan/dissectors/packet-isis-clv.c')
-rw-r--r-- | epan/dissectors/packet-isis-clv.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-isis-clv.c b/epan/dissectors/packet-isis-clv.c index 2863c9a3c6..77b9402d24 100644 --- a/epan/dissectors/packet-isis-clv.c +++ b/epan/dissectors/packet-isis-clv.c @@ -528,7 +528,7 @@ isis_dissect_clvs(tvbuff_t *tvb, packet_info* pinfo, proto_tree *tree, int offse code, length); proto_tree_add_item(clv_tree, tree_type, tvb, offset - 2, 1, ENC_BIG_ENDIAN); proto_tree_add_item(clv_tree, tree_length, tvb, offset - 1, 1, ENC_BIG_ENDIAN); - proto_tree_add_expert_format(clv_tree, pinfo, &ei_unknown, tvb, offset, length -2, "Dissector for IS-IS CLV (%d)" + proto_tree_add_expert_format(clv_tree, pinfo, &ei_unknown, tvb, offset, length, "Dissector for IS-IS CLV (%d)" " code not implemented, Contact Wireshark developers if you want this supported", code); } offset += length; |