diff options
| author | Jeff Morriss <jeff.morriss.ws@gmail.com> | 2013-06-30 16:59:58 +0000 |
|---|---|---|
| committer | Jeff Morriss <jeff.morriss.ws@gmail.com> | 2013-06-30 16:59:58 +0000 |
| commit | 7fa41b815c6fadac39c9d673ed138e5cac77560a (patch) | |
| tree | e8713cd2e2697ed7a815f8a32377f14fdb93e54c /plugins | |
| parent | c3aab70dd8a42d9a36d79794ab7f29a863f7d024 (diff) | |
Add a question about whether hf_pn_rt_frag_data should be FT_STRING or FT_BYTES.
svn path=/trunk/; revision=50261
Diffstat (limited to 'plugins')
| -rw-r--r-- | plugins/profinet/packet-pn-rt.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/plugins/profinet/packet-pn-rt.c b/plugins/profinet/packet-pn-rt.c index 4bb691c8ea..5d91b49d3a 100644 --- a/plugins/profinet/packet-pn-rt.c +++ b/plugins/profinet/packet-pn-rt.c @@ -408,6 +408,7 @@ dissect_FRAG_PDU_heur(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void uFragNumber, val_to_str( (u8FragStatus & 0x80) >> 7, pn_rt_frag_status_more_follows, "Unknown")); + /* Is this a string or a bunch of bytes? Should it be FT_BYTES? */ proto_tree_add_string_format(sub_tree, hf_pn_rt_frag_data, tvb, offset, tvb_length(tvb) - offset, "data", "Fragment Length: %d bytes", tvb_length(tvb) - offset); col_append_fstr(pinfo->cinfo, COL_INFO, " Fragment Length: %d bytes", tvb_length(tvb) - offset); @@ -949,6 +950,7 @@ proto_register_pn_rt(void) FT_UINT8, BASE_DEC, NULL, 0x3F, NULL, HFILL }}, + /* Is this a string or a bunch of bytes? Should it be FT_BYTES? */ { &hf_pn_rt_frag_data, { "FragData", "pn_rt.frag_data", FT_STRING, BASE_NONE, NULL, 0x00, |