]> www.infradead.org Git - users/dwmw2/linux.git/commitdiff
octeontx2-pf: fix netdev memory leak in rvu_rep_create()
authorHarshit Mogalapalli <harshit.m.mogalapalli@oracle.com>
Tue, 17 Dec 2024 05:23:24 +0000 (21:23 -0800)
committerJakub Kicinski <kuba@kernel.org>
Thu, 19 Dec 2024 03:23:50 +0000 (19:23 -0800)
When rvu_rep_devlink_port_register() fails, free_netdev(ndev) for this
incomplete iteration before going to "exit:" label.

Fixes: 9ed0343f561e ("octeontx2-pf: Add devlink port support")
Reviewed-by: Przemek Kitszel <przemyslaw.kitszel@intel.com>
Signed-off-by: Harshit Mogalapalli <harshit.m.mogalapalli@oracle.com>
Link: https://patch.msgid.link/20241217052326.1086191-1-harshit.m.mogalapalli@oracle.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/ethernet/marvell/octeontx2/nic/rep.c

index 232b10740c130a9c15ff7940f621511b7ec9764f..9e3fcbae5dee741826cd82638282e48b9109036b 100644 (file)
@@ -680,8 +680,10 @@ int rvu_rep_create(struct otx2_nic *priv, struct netlink_ext_ack *extack)
                ndev->features |= ndev->hw_features;
                eth_hw_addr_random(ndev);
                err = rvu_rep_devlink_port_register(rep);
-               if (err)
+               if (err) {
+                       free_netdev(ndev);
                        goto exit;
+               }
 
                SET_NETDEV_DEVLINK_PORT(ndev, &rep->dl_port);
                err = register_netdev(ndev);