diff options
author | Jeff Morriss <jeff.morriss@ulticom.com> | 2011-05-17 18:53:35 +0000 |
---|---|---|
committer | Jeff Morriss <jeff.morriss@ulticom.com> | 2011-05-17 18:53:35 +0000 |
commit | c204df4719a1d1da3fe8a0b573b68f5b88bce3a2 (patch) | |
tree | bb8c162458eef2a7e74c94e25b334e3ff6e367d7 /epan/dissectors/packet-xcsl.c | |
parent | ec460c3e993bf445a5a4270ee86b8e7a0ac48feb (diff) |
Use val_to_str() instead of blindly passing the return value from match_strval()
into a format routine (to ensure a non-NULL string pointer).
svn path=/trunk/; revision=37203
Diffstat (limited to 'epan/dissectors/packet-xcsl.c')
-rw-r--r-- | epan/dissectors/packet-xcsl.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-xcsl.c b/epan/dissectors/packet-xcsl.c index 9382bb0ed5..04bc55258a 100644 --- a/epan/dissectors/packet-xcsl.c +++ b/epan/dissectors/packet-xcsl.c @@ -67,7 +67,7 @@ static gint ett_xcsl = -1; #define XCSL_PROTSEQERR 8 #define XCSL_NONE 9 -/* Result code meanings. See function match_strval() for usage */ +/* Result code meanings. */ static const value_string xcsl_action_vals[] = { { XCSL_SUCCESS, "Success" }, { XCSL_UNKNOWN, "Unknown" }, @@ -218,7 +218,7 @@ static void dissect_xcsl_tcp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree if ( result >= XCSL_NONE ) { result = XCSL_UNDEFINED; } - code = match_strval(result, xcsl_action_vals); + code = val_to_str(result, xcsl_action_vals, "Unknown: %d"); /* Print result code and description */ xcsl_item = proto_tree_add_item(tree, hf_xcsl_result, tvb, offset, len, FALSE); |