diff options
author | Michael Mann <mmann78@netscape.net> | 2015-11-25 22:15:49 -0500 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2015-11-26 08:38:30 +0000 |
commit | be7bd77d993992c7bfa2aba43abf6cc9b5490cf5 (patch) | |
tree | abe55cc26de89d4c43b3e83cc5fc4a3b0131015c /epan/dissectors/packet-ppp.c | |
parent | a9f5d8503f85704bd6facc2de3f51f5d83f4862c (diff) |
Add BMT_NO_FLAGS for proto_tree_add_bitmask_XXX_with_flags.
This prevents some false positives for fix-encoding_args.pl which doesn't like constant values (like 0) passed as the last argument to any proto_tree_add_xxx call.
Change-Id: Iab7b09b425bf0d48126533daa0e65490bd7b8890
Reviewed-on: https://code.wireshark.org/review/12167
Reviewed-by: Michael Mann <mmann78@netscape.net>
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-ppp.c')
-rw-r--r-- | epan/dissectors/packet-ppp.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/epan/dissectors/packet-ppp.c b/epan/dissectors/packet-ppp.c index e386ad0e6d..020940d666 100644 --- a/epan/dissectors/packet-ppp.c +++ b/epan/dissectors/packet-ppp.c @@ -4503,7 +4503,7 @@ dissect_iphc_crtp_fh(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, void* proto_tree_add_bitmask_with_flags(fh_tree, tvb, 2, hf_iphc_crtp_fh_flags, ett_iphc_crtp_fh_flags, iphc_crtp_fh_flags_fields, ENC_BIG_ENDIAN, - 0); + BMT_NO_FLAGS); proto_tree_add_item(fh_tree, hf_iphc_crtp_gen, tvb, 2, 1, ENC_BIG_ENDIAN); @@ -4787,7 +4787,7 @@ dissect_iphc_crtp_cntcp(tvbuff_t *tvb, packet_info *pinfo, proto_tree *tree, voi } proto_tree_add_bitmask_with_flags(cntcp_tree, tvb, 1, hf_iphc_crtp_fh_flags, ett_iphc_crtp_fh_flags, iphc_crtp_fh_flags_fields, ENC_BIG_ENDIAN, - 0); + BMT_NO_FLAGS); proto_tree_add_item(cntcp_tree, hf_iphc_crtp_gen, tvb, 1, 1, ENC_BIG_ENDIAN); |