[PATCH 5.10 089/252]一个经典的内存泄漏场景

最新推荐文章于 2024-07-13 08:08:36 发布

velscode

最新推荐文章于 2024-07-13 08:08:36 发布

阅读量165

点赞数

CC 4.0 BY-SA版权

本文链接：https://blog.youkuaiyun.com/velscode/article/details/117440046

博客提到一个经典内存泄漏补丁，代码在drivers/net/usb/smsc75xx.c中，调用分配内存空间后，发生错误时未释放内存导致泄漏。修复方法是发生错误时不直接return，而是跳转到错误处理分支，释放内存后再return。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

https://www.spinics.net/lists/kernel/msg3959269.html

看到一个经典的内存泄漏的补丁

drivers/net/usb/smsc75xx.c

dev->data[0] = (unsigned long)kzalloc(sizeof(struct smsc75xx_priv),
					      GFP_KERNEL);

	pdata = (struct smsc75xx_priv *)(dev->data[0]);

backtrace:
  [<ffffffff84245b62>] kmalloc include/linux/slab.h:556 [inline]
  [<ffffffff84245b62>] kzalloc include/linux/slab.h:686 [inline]
  [<ffffffff84245b62>] smsc75xx_bind+0x7a/0x334 drivers/net/usb/smsc75xx.c:1460
  [<ffffffff82b5b2e6>] usbnet_probe+0x3b6/0xc30 drivers/net/usb/usbnet.c:1728

这段代码调用kzalloc分配了内存空间，用pdata指向。但是下面的代码里发生错误后没有free就进行了return造成内存泄漏

修复方法也很简单，就是发生错误时不直接return，而是通过goto跳转到错误处理分支，释放内存后再return。

--- a/drivers/net/usb/smsc75xx.c
+++ b/drivers/net/usb/smsc75xx.c
@@ -1483,7 +1483,7 @@ static int smsc75xx_bind(struct usbnet *
 	ret = smsc75xx_wait_ready(dev, 0);
 	if (ret < 0) {
 		netdev_warn(dev->net, "device not ready in smsc75xx_bind\n");
-		return ret;
+		goto err;
 	}
 
 	smsc75xx_init_mac_address(dev);
@@ -1492,7 +1492,7 @@ static int smsc75xx_bind(struct usbnet *
 	ret = smsc75xx_reset(dev);
 	if (ret < 0) {
 		netdev_warn(dev->net, "smsc75xx_reset error %d\n", ret);
-		return ret;
+		goto err;
 	}
 
 	dev->net->netdev_ops = &smsc75xx_netdev_ops;
@@ -1502,6 +1502,10 @@ static int smsc75xx_bind(struct usbnet *
 	dev->hard_mtu = dev->net->mtu + dev->net->hard_header_len;
 	dev->net->max_mtu = MAX_SINGLE_PACKET_SIZE;
 	return 0;
+
+err:
+	kfree(pdata);
+	return ret;
 }