staging: kpc2000: kpc_i2c: prevent memory leak in probe() error case
authorGeordan Neukum <gneukum1@gmail.com>
Sat, 18 May 2019 02:29:58 +0000 (02:29 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 20 May 2019 08:29:01 +0000 (10:29 +0200)
The probe() function performs a kzalloc to dynamically allocate memory
at runtime. If the allocation succeeds, yet invoking the function
i2c_add_adapter fails, the dynamically allocated memory is never freed.
Change the allocation to use the managed allocation API instead and
remove the manual freeing of the memory in the remove() function.

Signed-off-by: Geordan Neukum <gneukum1@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/kpc2000/kpc_i2c/i2c_driver.c

index 6cb63d20b00f64e404935ce4696663b37bd53869..9b9de81c8548ecaea8f8e37b206cdbe2053f28a4 100644 (file)
@@ -628,7 +628,7 @@ int pi2c_probe(struct platform_device *pldev)
 
        dev_dbg(&pldev->dev, "pi2c_probe(pldev = %p '%s')\n", pldev, pldev->name);
 
-       priv = kzalloc(sizeof(struct i2c_device), GFP_KERNEL);
+       priv = devm_kzalloc(&pldev->dev, sizeof(*priv), GFP_KERNEL);
        if (!priv) {
                return -ENOMEM;
        }
@@ -685,10 +685,6 @@ int pi2c_remove(struct platform_device *pldev)
        //pci_set_drvdata(dev, NULL);
 
        //cdev_del(&lddev->cdev);
-       if(lddev != 0) {
-               kfree(lddev);
-               pldev->dev.platform_data = 0;
-       }
 
        return 0;
 }