diff options
author | Guy Harris <guy@alum.mit.edu> | 2011-09-26 20:46:09 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2011-09-26 20:46:09 +0000 |
commit | 8e7403464ebde2d812f59a3e6a5eaad498f8fcad (patch) | |
tree | 924005d90d65c40da97560dc2ae0465d4f7c1059 | |
parent | e3779631d67c8f05c50aee5f089017c48a493488 (diff) |
The fragment data is almost certainly not a text string, so don't make
it an FT_STRING.
svn path=/trunk/; revision=39160
-rw-r--r-- | plugins/profinet/packet-pn-rt.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/plugins/profinet/packet-pn-rt.c b/plugins/profinet/packet-pn-rt.c index 92c427d4db..da43f74ec4 100644 --- a/plugins/profinet/packet-pn-rt.c +++ b/plugins/profinet/packet-pn-rt.c @@ -286,7 +286,8 @@ dissect_FRAG_PDU_heur(tvbuff_t *tvb, val_to_str( (u8FragStatus & 0x40) >> 6, pn_rt_frag_status_error, "Unknown")); - proto_tree_add_string_format(sub_tree, hf_pn_rt_frag_data, tvb, offset, tvb_length(tvb) - offset, "data", + /* XXX - should this use u8FragDataLength? */ + proto_tree_add_none_format(sub_tree, hf_pn_rt_frag_data, tvb, offset, tvb_length(tvb) - offset, "FragData: %d bytes", tvb_length(tvb) - offset); /* note: the actual defragmentation implementation is still missing here */ @@ -730,7 +731,7 @@ proto_register_pn_rt(void) { &hf_pn_rt_frag_status_fragment_number, { "FragmentNumber (zero based)", "pn_rt.frag_status.fragment_number", FT_UINT8, BASE_DEC, NULL, 0x3F, NULL, HFILL }}, { &hf_pn_rt_frag_data, { - "FragData", "pn_rt.frag_data", FT_STRING, BASE_NONE, NULL, 0x00, NULL, HFILL }}, + "FragData", "pn_rt.frag_data", FT_NONE, BASE_NONE, NULL, 0x00, NULL, HFILL }}, }; static gint *ett[] = { &ett_pn_rt, |