diff options
| author | Jeff Morriss <jeff.morriss.ws@gmail.com> | 2016-04-14 22:43:16 -0400 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2016-05-13 03:59:52 +0000 |
| commit | a1891f64ec7e56a38e61225ced912c65fd053793 (patch) | |
| tree | 81a4d35610fa9f89abfbcfc5a93d49aefac59e82 /epan/dissectors/asn1/inap | |
| parent | d12561f61a2569f9dab3c1415efc14075afe2b71 (diff) | |
Add filterable fields for each ISUP number.
Make a common number dissection function to do this without cut-n-pasting code
a whole bunch of times.
Clean up the number dissection code while we're there:
- Don't throw an exception when we found too many digits. Report it (expert
info) and move on.
- Avoid a useless (text) subtree: use the hf for the number as the root of
the tree. This means using proto_tree_append_string() which adds some
complication but it's worth it to avoid a hidden item (which was the old
solution if we passed the number to the E.164 dissector).
Bug: 12334
Change-Id: I465cab91b216b734f5763a4e56faa8e06b0731a2
Reviewed-on: https://code.wireshark.org/review/14924
Petri-Dish: Jeff Morriss <jeff.morriss.ws@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/asn1/inap')
| -rw-r--r-- | epan/dissectors/asn1/inap/inap.cnf | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/epan/dissectors/asn1/inap/inap.cnf b/epan/dissectors/asn1/inap/inap.cnf index a8565f81e7..cea4e082ed 100644 --- a/epan/dissectors/asn1/inap/inap.cnf +++ b/epan/dissectors/asn1/inap/inap.cnf @@ -234,7 +234,7 @@ static const inap_err_t inap_err_tab[] = { if (!parameter_tvb) return offset; -dissect_isup_called_party_number_parameter(parameter_tvb, tree, NULL); +dissect_isup_called_party_number_parameter(parameter_tvb, actx->pinfo, tree, NULL); #.FN_BODY CallingPartyNumber VAL_PTR = ¶meter_tvb tvbuff_t *parameter_tvb; @@ -244,7 +244,7 @@ dissect_isup_called_party_number_parameter(parameter_tvb, tree, NULL); if (!parameter_tvb) return offset; - dissect_isup_calling_party_number_parameter(parameter_tvb, tree, NULL); + dissect_isup_calling_party_number_parameter(parameter_tvb, actx->pinfo, tree, NULL); #.FN_BODY BearerCapability/bearerCap VAL_PTR = ¶meter_tvb @@ -289,7 +289,7 @@ dissect_isup_called_party_number_parameter(parameter_tvb, tree, NULL); if (!parameter_tvb) return offset; - dissect_isup_redirecting_number_parameter(parameter_tvb, tree, NULL); + dissect_isup_redirecting_number_parameter(parameter_tvb, actx->pinfo, tree, NULL); #.FN_BODY HighLayerCompatibility VAL_PTR = ¶meter_tvb /* |