diff options
| author | Michael Mann <mmann78@netscape.net> | 2013-09-30 15:21:09 +0000 |
|---|---|---|
| committer | Michael Mann <mmann78@netscape.net> | 2013-09-30 15:21:09 +0000 |
| commit | cf804429123802edb55003ebfd21ebea5718d10b (patch) | |
| tree | d2a468c4f0aeb706413a6c50c2e12026673a6b34 /epan/dissectors/packet-dcerpc-eventlog.c | |
| parent | 2211ab0fba17d71cc25f6b0d026f04d241bbced1 (diff) | |
Convert some proto_tree_add_string_format calls to something more appropriate.
There seem to be several cases of proto_tree_add_string_format where a "string" value/filter doesn't really make sense because it's always empty, and is just being used as a "filterable subtree header (placeholder)". They appear to be more for "presense" than "value" and should probably be FT_NONE, although I'd almost argue for removing the filter in favor of proto_tree_add_text.
svn path=/trunk/; revision=52296
Diffstat (limited to 'epan/dissectors/packet-dcerpc-eventlog.c')
| -rw-r--r-- | epan/dissectors/packet-dcerpc-eventlog.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-dcerpc-eventlog.c b/epan/dissectors/packet-dcerpc-eventlog.c index 55b906e444..8f36ec482c 100644 --- a/epan/dissectors/packet-dcerpc-eventlog.c +++ b/epan/dissectors/packet-dcerpc-eventlog.c @@ -408,7 +408,7 @@ eventlog_dissect_element_Record_strings(tvbuff_t *tvb, int offset, packet_info * int len; len=eventlog_get_unicode_string_length(tvb, string_offset); str=tvb_get_faked_unicode(wmem_packet_scope(), tvb, string_offset, len, TRUE); - proto_tree_add_string_format(tree, hf_eventlog_Record_string, tvb, string_offset, len*2, str, "string: %s", str); + proto_tree_add_string_format_value(tree, hf_eventlog_Record_string, tvb, string_offset, len*2, str, "%s", str); string_offset+=len*2; num_of_strings--; |