diff options
| author | Guy Harris <guy@alum.mit.edu> | 2003-04-02 08:31:37 +0000 |
|---|---|---|
| committer | Guy Harris <guy@alum.mit.edu> | 2003-04-02 08:31:37 +0000 |
| commit | 2e82b12e7c5da2d9e70b6a5afd2c6ad6d6b45d59 (patch) | |
| tree | d5a23ade6f88574a2ee0a6c21cdb62dd3d934872 /packet-nlsp.c | |
| parent | 56fdae20829b06efd116c5b0036e4883ceac8a8d (diff) | |
"proto_tree_add_text()" takes printf-style arguments; there's no need to
use snprintf to format a string into a buffer and then supply that
string to "proto_tree_add_text()".
svn path=/trunk/; revision=7395
Diffstat (limited to 'packet-nlsp.c')
| -rw-r--r-- | packet-nlsp.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/packet-nlsp.c b/packet-nlsp.c index b060113975..8981ed24f3 100644 --- a/packet-nlsp.c +++ b/packet-nlsp.c @@ -1,7 +1,7 @@ /* packet-nlsp.c * Routines for NetWare Link Services Protocol * - * $Id: packet-nlsp.c,v 1.3 2003/04/02 08:13:35 guy Exp $ + * $Id: packet-nlsp.c,v 1.4 2003/04/02 08:31:37 guy Exp $ * * Based on ISIS dissector by Stuart Stanley <stuarts@mxmail.net> * @@ -190,7 +190,6 @@ nlsp_dissect_clvs(tvbuff_t *tvb, proto_tree *tree, int offset, int q; proto_item *ti; proto_tree *clv_tree; - char sbuf[255]; int adj; while ( len > 0 ) { @@ -215,10 +214,9 @@ nlsp_dissect_clvs(tvbuff_t *tvb, proto_tree *tree, int offset, if ( opts[q].dissect ) { if (tree) { /* adjust by 2 for code/len octets */ - snprintf ( sbuf, sizeof(sbuf), "%s (%d)", - opts[q].tree_text, length ); ti = proto_tree_add_text(tree, tvb, offset - 2, - length + 2, sbuf); + length + 2, "%s (%u)", + opts[q].tree_text, length ); clv_tree = proto_item_add_subtree(ti, *opts[q].tree_id ); } else { @@ -228,10 +226,9 @@ nlsp_dissect_clvs(tvbuff_t *tvb, proto_tree *tree, int offset, length); } else { if (tree) { - snprintf ( sbuf, sizeof(sbuf), - "Unknown code (%d:%d)", code, length ); ti = proto_tree_add_text(tree, tvb, offset - 2, - length + 2, sbuf); + length + 2, "Unknown code %u (%u)", + code, length); clv_tree = proto_item_add_subtree(ti, unknown_tree_id ); } else { |