diff options
| author | Alexis La Goutte <alexis.lagoutte@gmail.com> | 2014-08-20 12:40:16 +0200 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2014-10-28 07:07:16 +0000 |
| commit | 25448b01bdf06ec8011afd9af9167e8081a42d0c (patch) | |
| tree | df85011e88d0cee951f670c631c1ed8c415bb7e2 /epan | |
| parent | 960ddb05b174681eaae2031df8c5285d44440e77 (diff) | |
| download | wireshark-25448b01bdf06ec8011afd9af9167e8081a42d0c.tar.gz | |
ISIS: Display code/type and length for unknown code/type
Change-Id: Idb70d571ae7d7f5fc87c08012ab380d35b2856a0
Reviewed-on: https://code.wireshark.org/review/4961
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-isis-clv.c | 13 |
1 files changed, 4 insertions, 9 deletions
diff --git a/epan/dissectors/packet-isis-clv.c b/epan/dissectors/packet-isis-clv.c index 6dc25b4bc7..cac9d3826c 100644 --- a/epan/dissectors/packet-isis-clv.c +++ b/epan/dissectors/packet-isis-clv.c @@ -552,17 +552,12 @@ isis_dissect_clvs(tvbuff_t *tvb, packet_info* pinfo, proto_tree *tree, int offse opts[q].dissect(tvb, pinfo, clv_tree, offset, id_length, length); } else { -#if 0 /* XXX: Left as commented out in case info about "unknown code" is ever to be displayed under a sub-tree */ clv_tree = proto_tree_add_subtree_format(tree, tvb, offset - 2, - length + 2, unknown_tree_id, NULL, "Unknown code %u (%u)", + length + 2, unknown_tree_id, NULL, "Unknown code (t=%u, l=%u)", code, length); -#else - if (tree) { - proto_tree_add_text(tree, tvb, offset - 2, - length + 2, "Unknown code %u (%u)", - code, length); - } -#endif + 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); + /* Add value with data display ? */ } offset += length; len -= length; |