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-isis-lsp.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-isis-lsp.c')
-rw-r--r-- | epan/dissectors/packet-isis-lsp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-isis-lsp.c b/epan/dissectors/packet-isis-lsp.c index 129c502549..ede2f3c972 100644 --- a/epan/dissectors/packet-isis-lsp.c +++ b/epan/dissectors/packet-isis-lsp.c @@ -2703,9 +2703,9 @@ isis_dissect_isis_lsp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, int o if (tree) { char* value = print_system_id( tvb_get_ptr(tvb, offset, id_length+2), id_length+2); - proto_tree_add_string_format(lsp_tree, hf_isis_lsp_lsp_id, + proto_tree_add_string_format_value(lsp_tree, hf_isis_lsp_lsp_id, tvb, offset, id_length + 2, - value, "LSP-ID: %s", value); + value, "%s", value); } col_append_fstr(pinfo->cinfo, COL_INFO, ", LSP-ID: %s", |