diff options
author | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2012-08-10 22:55:02 +0000 |
---|---|---|
committer | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2012-08-10 22:55:02 +0000 |
commit | eadd88560df528848a305d3c02d370e88a347fc9 (patch) | |
tree | 5bfcabbb83a6e7359cfc302cdc604264560d66a0 /epan/dissectors/packet-pppoe.c | |
parent | 43ca6a305349c1e81dc31e79016abe7743650d4b (diff) |
Use val_to_str_const() where appropriate;
Also (for a few files):
- create/use some extended value strings;
- remove unneeded #include files;
- remove unneeded variable initialization;
- re-order fcns slightly so prefs_reg_handoff...() at end, etc
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@44438 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-pppoe.c')
-rw-r--r-- | epan/dissectors/packet-pppoe.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/epan/dissectors/packet-pppoe.c b/epan/dissectors/packet-pppoe.c index 7855820eef..6aae01ff22 100644 --- a/epan/dissectors/packet-pppoe.c +++ b/epan/dissectors/packet-pppoe.c @@ -308,7 +308,7 @@ const value_string datarate_scale_vals[] = { #define CASE_VSPEC_DSLF_TAG_UINT(tag_name, relation, length, hf_var) case tag_name: \ if (!(poe_tag_length relation length)) { \ expert_add_info_format(pinfo, pppoe_tree, PI_MALFORMED, PI_WARN, "%s: Wrong length: %u (expected %s %d)", \ - val_to_str(poe_tag, vspec_tag_vals, "Unknown"), poe_tag_length, #relation, length); \ + val_to_str_const(poe_tag, vspec_tag_vals, "Unknown"), poe_tag_length, #relation, length); \ } else { \ proto_tree_add_item(pppoe_tree, hf_var, tvb, \ tagstart+2, poe_tag_length, ENC_BIG_ENDIAN); \ @@ -318,7 +318,7 @@ const value_string datarate_scale_vals[] = { #define CASE_VSPEC_DSLF_TAG_STRING(tag_name, relation, length, hf_var) case tag_name: \ if (!(poe_tag_length relation length)) { \ expert_add_info_format(pinfo, pppoe_tree, PI_MALFORMED, PI_WARN, "%s: Wrong length: %u (expected %s %d)", \ - val_to_str(poe_tag, vspec_tag_vals, "Unknown"), poe_tag_length, #relation, length); \ + val_to_str_const(poe_tag, vspec_tag_vals, "Unknown"), poe_tag_length, #relation, length); \ } else { \ proto_tree_add_item(pppoe_tree, hf_var, tvb, \ tagstart+2, poe_tag_length, ENC_ASCII|ENC_NA); \ @@ -399,7 +399,7 @@ dissect_pppoe_subtags_dslf(tvbuff_t *tvb, packet_info *pinfo _U_, int offset, pr if (poe_tag_length != 3) { expert_add_info_format(pinfo, ti, PI_MALFORMED, PI_WARN, "%s: Wrong length: %u (expected 3)", - val_to_str(poe_tag, vspec_tag_vals, "Unknown"), poe_tag_length); + val_to_str_const(poe_tag, vspec_tag_vals, "Unknown"), poe_tag_length); } encaps_tree = proto_item_add_subtree(ti, ett_pppoed_tag_vspec_dslf_access_loop_encaps); proto_tree_add_item(encaps_tree, hf_pppoed_tag_vspec_access_loop_encap_data_link, @@ -710,7 +710,7 @@ static void dissect_pppoed(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) if (check_col(pinfo->cinfo, COL_INFO)) { - col_append_str(pinfo->cinfo, COL_INFO, val_to_str(pppoe_code, code_vals, "Unknown")); + col_append_str(pinfo->cinfo, COL_INFO, val_to_str_const(pppoe_code, code_vals, "Unknown")); } /* Read length of payload */ @@ -1074,7 +1074,7 @@ static void dissect_pppoes(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree) if (check_col(pinfo->cinfo,COL_INFO)) { col_add_str(pinfo->cinfo, COL_INFO, - val_to_str(pppoe_code, code_vals, "Unknown")); + val_to_str_const(pppoe_code, code_vals, "Unknown")); } reported_payload_length = tvb_get_ntohs(tvb, 4); |