diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-07 20:47:11 +0200 |
---|---|---|
committer | Pascal Quantin <pascal.quantin@gmail.com> | 2015-06-07 19:00:33 +0000 |
commit | c100e1cf525a9bba61edd13850364c85f24522cc (patch) | |
tree | b4a51435d3300f58dcef4f4ce9acce8e580ed04a | |
parent | ae027ecd7cf2c7f35c9d7d710004b2199954c79e (diff) |
PER: fix V593 warning reported by PVS-Studio
Consider reviewing the expression of the 'A = B == C' kind. The expression is calculated as following: 'A = (B == C)'.
Change-Id: I162a2d081a70cb39b326d3aa2dc4108f49962169
Reviewed-on: https://code.wireshark.org/review/8821
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
-rw-r--r-- | epan/dissectors/packet-per.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-per.c b/epan/dissectors/packet-per.c index 4741a2b6d8..f1a523d13e 100644 --- a/epan/dissectors/packet-per.c +++ b/epan/dissectors/packet-per.c @@ -2565,7 +2565,7 @@ call_per_oid_callback(const char *oid, tvbuff_t *tvb, packet_info *pinfo, proto_ val_tvb = new_octet_aligned_subset(tvb, offset, actx, type_length); if (oid == NULL || - (len = dissector_try_string(per_oid_dissector_table, oid, val_tvb, pinfo, tree, actx) == 0)) + ((len = dissector_try_string(per_oid_dissector_table, oid, val_tvb, pinfo, tree, actx)) == 0)) { proto_tree_add_expert(tree, pinfo, &ei_per_oid_not_implemented, val_tvb, 0, -1); dissect_per_open_type(tvb, start_offset, actx, tree, hf_index, NULL); |