Thread (2 messages) 2 messages, 2 authors, 3d ago
WARM3d

[PATCH v3 net-next] net: rnpgbe: Delete a null pointer check in rnpgbe_rm_adapter()

From: Markus Elfring <hidden>
Date: 2026-07-17 08:44:57
Also in: kernel-janitors, lkml
Subsystem: mucse ethernet driver, networking drivers, the rest · Maintainers: Yibo Dong, Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds

From: Markus Elfring <redacted>
Date: Fri, 17 Jul 2026 10:20:15 +0200

The check for the pointer “mucse” was identified as redundant because
the relevant address information was preserved according to calls of
the functions “pci_set_drvdata” and “pci_get_drvdata”.
Thus remove it.

This issue was detected by using the Coccinelle software.

See also:
Reconsidering pointer dereferences before null pointer checks (with SmPL)
2023-04-09
https://lore.kernel.org/cocci/1a11455f-ab57-dce0-1677-6beb8492a257@web.de/ (local)
https://sympa.inria.fr/sympa/arc/cocci/2023-04/msg00021.html

Signed-off-by: Markus Elfring <redacted>
---

v3:
Andrew Lunn and Julia Lawall requested an extended change description.

v2:
Andrew Lunn requested the removal of a null pointer check.


 drivers/net/ethernet/mucse/rnpgbe/rnpgbe_main.c | 2 --
 1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/mucse/rnpgbe/rnpgbe_main.c b/drivers/net/ethernet/mucse/rnpgbe/rnpgbe_main.c
index 70a2b0082ba8..82a3fbfb3c0a 100644
--- a/drivers/net/ethernet/mucse/rnpgbe/rnpgbe_main.c
+++ b/drivers/net/ethernet/mucse/rnpgbe/rnpgbe_main.c
@@ -249,8 +249,6 @@ static void rnpgbe_rm_adapter(struct pci_dev *pdev)
 	struct net_device *netdev;
 	int err;
 
-	if (!mucse)
-		return;
 	netdev = mucse->netdev;
 	unregister_netdev(netdev);
 	err = rnpgbe_send_notify(hw, false, mucse_fw_powerup);
-- 
2.55.0
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help