ethtool: Fix an error code in cxgb2.c
authorYang Li <yang.lee@linux.alibaba.com>
Fri, 3 Sep 2021 06:42:33 +0000 (14:42 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 22 Sep 2021 09:45:34 +0000 (11:45 +0200)
[ Upstream commit 7db8263a12155c7ae4ad97e850f1e499c73765fc ]

When adapter->registered_device_map is NULL, the value of err is
uncertain, we set err to -EINVAL to avoid ambiguity.

Clean up smatch warning:
drivers/net/ethernet/chelsio/cxgb/cxgb2.c:1114 init_one() warn: missing
error code 'err'

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/ethernet/chelsio/cxgb/cxgb2.c

index 8623be13bf86f2cc82f579daadc8121dcfb8432d..eef8fa1008890b9c7cf753b586027fab8dbb746c 100644 (file)
@@ -1157,6 +1157,7 @@ static int init_one(struct pci_dev *pdev, const struct pci_device_id *ent)
        if (!adapter->registered_device_map) {
                pr_err("%s: could not register any net devices\n",
                       pci_name(pdev));
+               err = -EINVAL;
                goto out_release_adapter_res;
        }