diff options
| author | Martin Kaiser <wireshark@kaiser.cx> | 2015-07-12 12:27:08 +0200 |
|---|---|---|
| committer | Martin Kaiser <wireshark@kaiser.cx> | 2015-07-12 10:29:28 +0000 |
| commit | 71190bca22189f678f164a81a35ac19feec34724 (patch) | |
| tree | 122c663d85d2b18a0c75920c80efc65b09dd8831 /epan | |
| parent | 8c2fc911dfe6b3296bbd2d7e404b15fd55a3b3a4 (diff) | |
megaco: replace THROW() with an expert info + return
Change-Id: Ib876ad4b0c6d700da1eec8c8512225c123ad14da
Reviewed-on: https://code.wireshark.org/review/9604
Reviewed-by: Martin Kaiser <wireshark@kaiser.cx>
Diffstat (limited to 'epan')
| -rw-r--r-- | epan/dissectors/packet-megaco.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/epan/dissectors/packet-megaco.c b/epan/dissectors/packet-megaco.c index e822245059..75050b2f8e 100644 --- a/epan/dissectors/packet-megaco.c +++ b/epan/dissectors/packet-megaco.c @@ -3101,8 +3101,10 @@ dissect_megaco_LocalControldescriptor(tvbuff_t *tvb, proto_tree *megaco_mediades token_index = find_megaco_localParam_names(tvb, tvb_current_offset, token_name_len); /* Find start of parameter value */ tvb_offset = tvb_find_guint8(tvb, tvb_offset , tvb_next_offset, '='); - if (tvb_offset == -1) - THROW(ReportedBoundsError); + if (tvb_offset == -1) { + expert_add_info(pinfo, megaco_LocalControl_item, &ei_megaco_parse_error); + return; + } /* Start search after '=' in case there is no SP*/ tvb_offset++; tvb_current_offset = megaco_tvb_skip_wsp(tvb, tvb_offset); |