diff options
author | gerald <gerald@f5534014-38df-0310-8fa8-9805f1628bb7> | 2008-02-16 01:33:48 +0000 |
---|---|---|
committer | gerald <gerald@f5534014-38df-0310-8fa8-9805f1628bb7> | 2008-02-16 01:33:48 +0000 |
commit | 452d172115aa96927d22db73d8683d9950cb3774 (patch) | |
tree | 776c273edc0474572d87235a8c81d6072a47e450 /epan/dissectors/packet-tftp.c | |
parent | ef32a2caefdb4bef06d0815222e14aa021cbf750 (diff) |
Format our text before displaying it. This keeps Wireshark from crashing on
Ubuntu 7.10 and possibly other platforms.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@24341 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-tftp.c')
-rw-r--r-- | epan/dissectors/packet-tftp.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/epan/dissectors/packet-tftp.c b/epan/dissectors/packet-tftp.c index c892131326..92fb4e2929 100644 --- a/epan/dissectors/packet-tftp.c +++ b/epan/dissectors/packet-tftp.c @@ -124,8 +124,8 @@ tftp_dissect_options(tvbuff_t *tvb, packet_info *pinfo, int offset, option_len = tvb_strsize(tvb, offset); /* length of option */ value_offset = offset + option_len; value_len = tvb_strsize(tvb, value_offset); /* length of value */ - optionname = tvb_get_ptr(tvb, offset, option_len); - optionvalue = tvb_get_ptr(tvb, value_offset, value_len); + optionname = tvb_format_text(tvb, offset, option_len); + optionvalue = tvb_format_text(tvb, value_offset, value_len); opt_item = proto_tree_add_text(tree, tvb, offset, option_len+value_len, "Option: %s = %s", optionname, optionvalue); @@ -220,7 +220,7 @@ dissect_tftp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) if (check_col(pinfo->cinfo, COL_INFO)) { - col_add_str(pinfo->cinfo, COL_INFO, + col_add_str(pinfo->cinfo, COL_INFO, val_to_str(opcode, tftp_opcode_vals, "Unknown (0x%04x)")); } @@ -245,7 +245,7 @@ dissect_tftp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) } if (check_col(pinfo->cinfo, COL_INFO)) { col_append_fstr(pinfo->cinfo, COL_INFO, ", File: %s", - tvb_get_ptr(tvb, offset, i1)); + tvb_format_text(tvb, offset, i1)); } offset += i1; @@ -256,7 +256,7 @@ dissect_tftp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) } if (check_col(pinfo->cinfo, COL_INFO)) { col_append_fstr(pinfo->cinfo, COL_INFO, ", Transfer type: %s", - tvb_get_ptr(tvb, offset, i1)); + tvb_format_text(tvb, offset, i1)); } offset += i1; @@ -273,7 +273,7 @@ dissect_tftp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) } if (check_col(pinfo->cinfo, COL_INFO)) { col_append_fstr(pinfo->cinfo, COL_INFO, ", File: %s", - tvb_get_ptr(tvb, offset, i1)); + tvb_format_text(tvb, offset, i1)); } offset += i1; @@ -284,7 +284,7 @@ dissect_tftp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) } if (check_col(pinfo->cinfo, COL_INFO)) { col_append_fstr(pinfo->cinfo, COL_INFO, ", Transfer type: %s", - tvb_get_ptr(tvb, offset, i1)); + tvb_format_text(tvb, offset, i1)); } offset += i1; @@ -348,7 +348,7 @@ dissect_tftp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) } if (check_col(pinfo->cinfo, COL_INFO)) { col_append_fstr(pinfo->cinfo, COL_INFO, ", Message: %s", - tvb_get_ptr(tvb, offset, i1)); + tvb_format_text(tvb, offset, i1)); } expert_add_info_format(pinfo, NULL, PI_RESPONSE_CODE, PI_NOTE, "TFTP blocksize out of range"); |