From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gu Zheng Subject: [PATCH] i2c: fix a potential kmemleak of adapter device Date: Wed, 18 Dec 2013 09:18:08 +0800 Message-ID: <52B0F7D0.2020506@cn.fujitsu.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Sender: linux-i2c-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: wsa-z923LK4zBo2bacvFa/9K2g@public.gmane.org Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel List-Id: linux-i2c@vger.kernel.org When running with the latest kernel, we get the following kmemleak message: unreferenced object 0xffff8800c2a36100 (size 256): comm "modprobe", pid 629, jiffies 4294676002 (age 1531.115s) hex dump (first 32 bytes): 00 00 00 00 ad 4e ad de ff ff ff ff 00 00 00 00 .....N.......... ff ff ff ff ff ff ff ff 78 21 85 82 ff ff ff ff ........x!...... backtrace: [] kmemleak_alloc+0x73/0x98 [] slab_post_alloc_hook+0x28/0x2a [] __kmalloc+0x12c/0x158 [] kzalloc.constprop.6+0xe/0x10 [] device_private_init+0x18/0x66 [] dev_set_drvdata+0x1e/0x34 [] i801_probe+0x5d/0x447 [i2c_i801] [] local_pci_probe+0x41/0x84 [] pci_device_probe+0xdf/0x10c [] driver_probe_device+0x12f/0x2ee [] __driver_attach+0x5f/0x83 [] bus_for_each_dev+0x64/0x90 [] driver_attach+0x1e/0x20 [] bus_add_driver+0xf2/0x1f0 [] driver_register+0x8c/0xc3 [] __pci_register_driver+0x62/0x67 According to the backtrace, it seems that we do not release the device_private field of adapter device in the fail path of i801_probe or the flow of i2c_del_adapter, so add the cleanup step to fix it. Signed-off-by: Gu Zheng --- drivers/i2c/busses/i2c-i801.c | 1 + drivers/i2c/i2c-core.c | 1 + 2 files changed, 2 insertions(+), 0 deletions(-) diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c index 737e298..4bd4fba 100644 --- a/drivers/i2c/busses/i2c-i801.c +++ b/drivers/i2c/busses/i2c-i801.c @@ -1246,6 +1246,7 @@ exit_free_irq: exit_release: pci_release_region(dev, SMBBAR); exit: + kfree(priv->adapter.dev.p); kfree(priv); return err; } diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c index d74c0b3..a7bab9a 100644 --- a/drivers/i2c/i2c-core.c +++ b/drivers/i2c/i2c-core.c @@ -1491,6 +1491,7 @@ void i2c_del_adapter(struct i2c_adapter *adap) idr_remove(&i2c_adapter_idr, adap->nr); mutex_unlock(&core_lock); + kfree(adap->dev.p); /* Clear the device structure in case this adapter is ever going to be added again */ memset(&adap->dev, 0, sizeof(adap->dev)); -- 1.7.7