diff options
| author | Michael Mann <mmann78@netscape.net> | 2014-09-06 18:04:28 -0400 |
|---|---|---|
| committer | Evan Huus <eapache@gmail.com> | 2014-09-08 15:46:05 +0000 |
| commit | cf8f11f67876462510cc4254b845063c6cc047de (patch) | |
| tree | c9fc47b3795bbb7bbb88c22a7c4014978342bd69 /epan/dissectors/packet-t30.c | |
| parent | 433a444d148f86f2562f804d25a57d00dc277cc0 (diff) | |
Eliminate proto_tree_add_text from some of the dissectors.
Other minor cleanups while in the area.
Change-Id: I623d941e53128f169e55dfc629547b4221fa72fc
Reviewed-on: https://code.wireshark.org/review/4021
Reviewed-by: Michael Mann <mmann78@netscape.net>
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Evan Huus <eapache@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-t30.c')
| -rw-r--r-- | epan/dissectors/packet-t30.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-t30.c b/epan/dissectors/packet-t30.c index 0169ee6ade..ea4a057e49 100644 --- a/epan/dissectors/packet-t30.c +++ b/epan/dissectors/packet-t30.c @@ -523,8 +523,8 @@ dissect_t30_numbers(tvbuff_t *tvb, int offset, packet_info *pinfo, int len, prot g_snprintf(t38->desc, MAX_T38_DESC, "Num: %s", str_num); } else { - proto_tree_add_text(tree, tvb, offset, tvb_reported_length_remaining(tvb, offset), - "[MALFORMED OR SHORT PACKET: number of digits must be 20]"); + proto_tree_add_expert_format(tree, pinfo, &ei_t30_bad_length, tvb, offset, -1, + "MALFORMED OR SHORT PACKET: number of digits must be 20"); col_append_str(pinfo->cinfo, COL_INFO, " [MALFORMED OR SHORT PACKET: number of digits must be 20]" ); } |