[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <YKIwFAmmm2W2XocO@mwanda>
Date: Mon, 17 May 2021 11:57:56 +0300
From: Dan Carpenter <dan.carpenter@...cle.com>
To: Chris Snook <chris.snook@...il.com>,
Johannes Berg <johannes@...solutions.net>
Cc: Zekun Shen <bruceshenzk@...il.com>, netdev@...r.kernel.org,
kernel-janitors@...r.kernel.org,
"David S. Miller" <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>
Subject: [PATCH 1/2 net-next] alx: fix a double unlock in alx_probe()
We're not holding the lock at this point so "goto unlock;" should be
"goto unmap;"
Fixes: 4a5fe57e7751 ("alx: use fine-grained locking instead of RTNL")
Signed-off-by: Dan Carpenter <dan.carpenter@...cle.com>
---
drivers/net/ethernet/atheros/alx/main.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/atheros/alx/main.c b/drivers/net/ethernet/atheros/alx/main.c
index e8884879a50f..45e380f3b065 100644
--- a/drivers/net/ethernet/atheros/alx/main.c
+++ b/drivers/net/ethernet/atheros/alx/main.c
@@ -1859,7 +1859,7 @@ static int alx_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
err = register_netdev(netdev);
if (err) {
dev_err(&pdev->dev, "register netdevice failed\n");
- goto out_unlock;
+ goto out_unmap;
}
netdev_info(netdev,
--
2.30.2
Powered by blists - more mailing lists