]> git.baikalelectronics.ru Git - kernel.git/commitdiff
mISDN: fix possible memory leak in mISDN_register_device()
authorYang Yingliang <yangyingliang@huawei.com>
Mon, 31 Oct 2022 12:13:40 +0000 (20:13 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 2 Nov 2022 12:34:48 +0000 (12:34 +0000)
Afer commit 1fa5ae857bb1 ("driver core: get rid of struct device's
bus_id string array"), the name of device is allocated dynamically,
add put_device() to give up the reference, so that the name can be
freed in kobject_cleanup() when the refcount is 0.

Set device class before put_device() to avoid null release() function
WARN message in device_release().

Fixes: 1fa5ae857bb1 ("driver core: get rid of struct device's bus_id string array")
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/isdn/mISDN/core.c

index a41b4b264594121e45b338d151c59fd4937c16c4..7ea0100f218a04890426c830baa1fe4bec7c0a73 100644 (file)
@@ -233,11 +233,12 @@ mISDN_register_device(struct mISDNdevice *dev,
        if (debug & DEBUG_CORE)
                printk(KERN_DEBUG "mISDN_register %s %d\n",
                       dev_name(&dev->dev), dev->id);
+       dev->dev.class = &mISDN_class;
+
        err = create_stack(dev);
        if (err)
                goto error1;
 
-       dev->dev.class = &mISDN_class;
        dev->dev.platform_data = dev;
        dev->dev.parent = parent;
        dev_set_drvdata(&dev->dev, dev);
@@ -249,8 +250,8 @@ mISDN_register_device(struct mISDNdevice *dev,
 
 error3:
        delete_stack(dev);
-       return err;
 error1:
+       put_device(&dev->dev);
        return err;
 
 }