diff options
author | Colin Ian King <colin.king@canonical.com> | 2017-05-07 14:05:16 -0400 |
---|---|---|
committer | Corey Minyard <cminyard@mvista.com> | 2017-05-17 19:30:35 -0400 |
commit | bf10ff69dd6e27710b21863ebd8e6504d9516222 (patch) | |
tree | bf9dd29df979103926dfb93dcfae251fe491e0da | |
parent | 860f01e96981a68553f3ca49f574ff14fe955e72 (diff) |
ipmi_ssif: remove redundant null check on array client->adapter->name
The null check on client->adapter->name is redundant as name is an
array of I2C_NAME_SIZE chars and hence can never be null. We may as
well remove this redundant check.
Detected by CoverityScan, CID#1375918 ("Array compared against 0")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Corey Minyard <cminyard@mvista.com>
-rw-r--r-- | drivers/char/ipmi/ipmi_ssif.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/char/ipmi/ipmi_ssif.c b/drivers/char/ipmi/ipmi_ssif.c index 6dd6476ea5d3..1d4fd846e457 100644 --- a/drivers/char/ipmi/ipmi_ssif.c +++ b/drivers/char/ipmi/ipmi_ssif.c | |||
@@ -1419,8 +1419,7 @@ static int find_slave_address(struct i2c_client *client, int slave_addr) | |||
1419 | list_for_each_entry(info, &ssif_infos, link) { | 1419 | list_for_each_entry(info, &ssif_infos, link) { |
1420 | if (info->binfo.addr != client->addr) | 1420 | if (info->binfo.addr != client->addr) |
1421 | continue; | 1421 | continue; |
1422 | if (info->adapter_name && client->adapter->name && | 1422 | if (info->adapter_name && strcmp_nospace(info->adapter_name, |
1423 | strcmp_nospace(info->adapter_name, | ||
1424 | client->adapter->name)) | 1423 | client->adapter->name)) |
1425 | continue; | 1424 | continue; |
1426 | if (info->slave_addr) { | 1425 | if (info->slave_addr) { |