diff options
author | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2009-08-28 18:37:53 +0000 |
---|---|---|
committer | wmeier <wmeier@f5534014-38df-0310-8fa8-9805f1628bb7> | 2009-08-28 18:37:53 +0000 |
commit | 5e356c514f66992bcf9b6c362864097da4d8ad90 (patch) | |
tree | 2d13e7f3db63188811531fe7f68ffcf6abbe3426 /epan/dissectors/packet-ipmi-transport.c | |
parent | c88208fa2c3a309e9b7025459daee3132d94bff7 (diff) |
Fix if test in fcn rs11():
return if either (as opposed to both) of the required sparameters is not present;
(Found by clang scan-build).
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@29600 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'epan/dissectors/packet-ipmi-transport.c')
-rw-r--r-- | epan/dissectors/packet-ipmi-transport.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/epan/dissectors/packet-ipmi-transport.c b/epan/dissectors/packet-ipmi-transport.c index df8db494d1..ca56e42139 100644 --- a/epan/dissectors/packet-ipmi-transport.c +++ b/epan/dissectors/packet-ipmi-transport.c @@ -1853,7 +1853,7 @@ rs11(tvbuff_t *tvb, proto_tree *tree) proto_tree_add_bitmask_text(tree, tvb, 0, 1, NULL, NULL, ett_ipmi_trn_11_rev, byte1, TRUE, 0); - if (!ipmi_getsaveddata(0, &pno) && !ipmi_getsaveddata(1, &req)) { + if (!ipmi_getsaveddata(0, &pno) || !ipmi_getsaveddata(1, &req)) { /* No request found - cannot parse further */ if (tvb_length(tvb) > 1) { proto_tree_add_item(tree, hf_ipmi_trn_11_param_data, tvb, 1, tvb_length(tvb) - 1, TRUE); |