diff options
author | AndersBroman <anders.broman@ericsson.com> | 2014-06-30 16:04:50 +0200 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2014-06-30 14:14:14 +0000 |
commit | 9387979e9136deaea43db2a2157d5e53f00f86ec (patch) | |
tree | d66de52fe99f05eb8630aa44f90cc2c55daac00f /epan | |
parent | 099bc9342f825a8d970bb888f16a5c0e271686ae (diff) |
Fix the display of AVP:s that are not fully dissected
fixes Bug #10237
Change-Id: I3c38bb4af4440f1e5087f21ef022bf2cd3a66002
Reviewed-on: https://code.wireshark.org/review/2720
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan')
-rw-r--r-- | epan/dissectors/packet-l2tp.c | 31 |
1 files changed, 16 insertions, 15 deletions
diff --git a/epan/dissectors/packet-l2tp.c b/epan/dissectors/packet-l2tp.c index ebec397251..d8cfb47306 100644 --- a/epan/dissectors/packet-l2tp.c +++ b/epan/dissectors/packet-l2tp.c @@ -651,23 +651,23 @@ static const value_string avp_type_vals[] = { { 83, "Withdraw Outgoing Sessions" }, /*[RFC4045] */ { 84, "Multicast Packets Priority" }, /*[RFC4045] */ { 85, "Frame-Relay Header Length" }, /*[RFC4591] */ - { 86, "ATM Maximum Concatenated Cells AVP" }, /*[RFC4454] */ - { 87, "OAM Emulation Required AVP" }, /*[RFC4454] */ - { 88, "ATM Alarm Status AVP" }, /*[RFC4454] */ + { 86, "ATM Maximum Concatenated Cells" }, /*[RFC4454] */ + { 87, "OAM Emulation Required" }, /*[RFC4454] */ + { 88, "ATM Alarm Status" }, /*[RFC4454] */ /* Also, see ATM Alarm Status AVP Values below */ { 89, "Attachment Group Identifier" }, /*[RFC4667] */ { 90, "Local End Identifier" }, /*[RFC4667] */ { 91, "Interface Maximum Transmission Unit" }, /*[RFC4667] */ { 92, "FCS Retention" }, /*[RFC4720] */ - { 93, "Tunnel Switching Aggregator ID AVP" }, /*[draft-ietf-l2tpext-tunnel-switching-06.txt] */ - { 94, "Maximum Receive Unit (MRU) AVP" }, /*[RFC4623] */ - { 95, "Maximum Reassembled Receive Unit (MRRU) AVP" }, /*[RFC4623] */ - { 96, "VCCV Capability AVP" }, /*[RFC5085] */ - { 97, "Connect Speed Update AVP" }, /*[RFC5515] */ - { 98, "Connect Speed Update Enable AVP" }, /*[RFC5515] */ - { 99, "TDM Pseudowire AVP" }, /*[RFC5611] */ + { 93, "Tunnel Switching Aggregator ID" }, /*[draft-ietf-l2tpext-tunnel-switching-06.txt] */ + { 94, "Maximum Receive Unit (MRU)" }, /*[RFC4623] */ + { 95, "Maximum Reassembled Receive Unit (MRRU)" }, /*[RFC4623] */ + { 96, "VCCV Capability" }, /*[RFC5085] */ + { 97, "Connect Speed Update" }, /*[RFC5515] */ + { 98, "Connect Speed Update Enable" }, /*[RFC5515] */ + { 99, "TDM Pseudowire" }, /*[RFC5611] */ { 100, "RTP AVP" }, /*[RFC5611] */ - { 101, "PW Switching Point AVP" }, /*[RFC6073] */ + { 101, "PW Switching Point" }, /*[RFC6073] */ { 0, NULL } }; @@ -1390,7 +1390,7 @@ static int dissect_l2tp_cisco_avps(tvbuff_t *tvb, packet_info *pinfo _U_, proto_ default: proto_tree_add_text(l2tp_avp_tree, tvb, offset, - avp_len, "Vendor-Specific AVP"); + avp_len-6, "Vendor-Specific AVP data"); break; } offset += avp_len; @@ -1557,7 +1557,7 @@ static void process_control_avps(tvbuff_t *tvb, proto_tree_add_text(l2tp_avp_tree, tvb, idx, 2, "Type: %u", avp_type); idx += 2; - proto_tree_add_text(l2tp_avp_tree, tvb, idx, avp_len, "Vendor-Specific AVP"); + proto_tree_add_text(l2tp_avp_tree, tvb, idx, avp_len-6, "Vendor-Specific AVP data"); avp_len-=6; } idx += avp_len; @@ -2006,8 +2006,9 @@ static void process_control_avps(tvbuff_t *tvb, break; } default: - proto_tree_add_text(l2tp_avp_tree, tvb, idx, avp_len, - "Unknown AVP"); + if(avp_len>0) + proto_tree_add_text(l2tp_avp_tree, tvb, idx, avp_len, + "AVP data not dissected yet"); break; } |