diff options
| author | Bruno Verstuyft <bruno.verstuyft@excentis.com> | 2019-06-20 15:18:42 +0200 |
|---|---|---|
| committer | Martin Kaiser <wireshark@kaiser.cx> | 2019-06-20 18:11:36 +0000 |
| commit | f9225faa87de80af5374f5fb103364b6cef5bc63 (patch) | |
| tree | 5f12cc6954551e46748751795e67841136ddace2 /epan | |
| parent | 44594aeb0dbcbe4199112190df44f455db565006 (diff) | |
DOCSIS: REG-ACK: do not show TLV Data if not present
Change-Id: I655b5b9eb75d339afbd8b9184ef85ffb7dd33774
Reviewed-on: https://code.wireshark.org/review/33681
Petri-Dish: Anders Broman <a.broman58@gmail.com>
Tested-by: Petri Dish Buildbot
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-docsis-macmgmt.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/epan/dissectors/packet-docsis-macmgmt.c b/epan/dissectors/packet-docsis-macmgmt.c index 22bedd9178..8cc32aba2d 100644 --- a/epan/dissectors/packet-docsis-macmgmt.c +++ b/epan/dissectors/packet-docsis-macmgmt.c @@ -3949,8 +3949,12 @@ dissect_regack (tvbuff_t * tvb, packet_info * pinfo, proto_tree * tree, void* da val_to_str_ext (response, &docsis_conf_code_ext, "%d")); /* Call Dissector for Appendix C TLV's */ - next_tvb = tvb_new_subset_remaining (tvb, 3); - call_dissector (docsis_tlv_handle, next_tvb, pinfo, regack_tree); + if(tvb_reported_length_remaining(tvb, 3) > 0 ) + { + next_tvb = tvb_new_subset_remaining (tvb, 3); + call_dissector (docsis_tlv_handle, next_tvb, pinfo, regack_tree); + } + return tvb_captured_length(tvb); } |