diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-07 16:16:37 +0200 |
---|---|---|
committer | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-07 14:35:27 +0000 |
commit | cefe9d3c26825ef848f6e6aedbec572db002f8b1 (patch) | |
tree | 09499981219a9c72400b4aabcef6f22c9182cf35 /epan/dissectors/packet-ecp-oui.c | |
parent | 83351a9996b04a5b3b2e7f3f705ffc63d13a0b53 (diff) |
ECP: fix V562 warning reported by PVS-Studio
It's odd to compare 0 or 1 with a value of 1: ((entries > 1)) == 1.
Change-Id: I6261389dddbbd7e60e98b8c351150d491f9cbddb
Reviewed-on: https://code.wireshark.org/review/8810
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Diffstat (limited to 'epan/dissectors/packet-ecp-oui.c')
-rw-r--r-- | epan/dissectors/packet-ecp-oui.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-ecp-oui.c b/epan/dissectors/packet-ecp-oui.c index 06325843a5..a21326c16a 100644 --- a/epan/dissectors/packet-ecp-oui.c +++ b/epan/dissectors/packet-ecp-oui.c @@ -165,7 +165,7 @@ dissect_vdp_fi_macvid(tvbuff_t *tvb, packet_info *pinfo _U_, proto_tree *tree, g entries = tvb_get_ntohs(tvb, offset); ecp_vdp_tlv_fi_subtree = proto_tree_add_subtree_format(tree, tvb, tempOffset, 2, ett_ecp, NULL, - "%i MAC/VID pair%s", entries, plurality((entries > 1), "s", "")); + "%i MAC/VID pair%s", entries, plurality(entries, "", "s")); tempOffset += 2; |