diff options
author | Denis Plotnikov <den-plotnikov@yandex-team.ru> | 2023-04-07 10:18:49 +0300 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2023-04-12 08:32:38 +0100 |
commit | 7573099e10ca69c3be33995c1fcd0d241226816d (patch) | |
tree | 396dfcc4af89054ef2bd3017f3731d2eaee5d42d /net/smc | |
parent | adacf21f1c9a65e299a103f78b3e6dd71c5ec99b (diff) | |
download | lwn-7573099e10ca69c3be33995c1fcd0d241226816d.tar.gz lwn-7573099e10ca69c3be33995c1fcd0d241226816d.zip |
qlcnic: check pci_reset_function result
Static code analyzer complains to unchecked return value.
The result of pci_reset_function() is unchecked.
Despite, the issue is on the FLR supported code path and in that
case reset can be done with pcie_flr(), the patch uses less invasive
approach by adding the result check of pci_reset_function().
Found by Linux Verification Center (linuxtesting.org) with SVACE.
Fixes: 7e2cf4feba05 ("qlcnic: change driver hardware interface mechanism")
Signed-off-by: Denis Plotnikov <den-plotnikov@yandex-team.ru>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Reviewed-by: Bjorn Helgaas <bhelgaas@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/smc')
0 files changed, 0 insertions, 0 deletions