]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
mei: Fix error handling in mei_register
authorMa Ke <make24@iscas.ac.cn>
Tue, 4 Nov 2025 02:01:33 +0000 (10:01 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 26 Nov 2025 12:31:44 +0000 (13:31 +0100)
mei_register() fails to release the device reference in error paths
after device_initialize(). During normal device registration, the
reference is properly handled through mei_deregister() which calls
device_destroy(). However, in error handling paths (such as cdev_alloc
failure, cdev_add failure, etc.), missing put_device() calls cause
reference count leaks, preventing the device's release function
(mei_device_release) from being called and resulting in memory leaks
of mei_device.

Found by code review.

Cc: stable <stable@kernel.org>
Fixes: 7704e6be4ed2 ("mei: hook mei_device on class device")
Signed-off-by: Ma Ke <make24@iscas.ac.cn>
Acked-by: Alexander Usyskin <alexander.usyskin@intel.com>
Link: https://patch.msgid.link/20251104020133.5017-1-make24@iscas.ac.cn
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/mei/main.c

index 86a73684a373253d4dc88f0fe43d6cf07d455b23..6f26d516078835f4454e53bb3daf49f613eb933c 100644 (file)
@@ -1307,6 +1307,7 @@ int mei_register(struct mei_device *dev, struct device *parent)
 err_del_cdev:
        cdev_del(dev->cdev);
 err:
+       put_device(&dev->dev);
        mei_minor_free(minor);
        return ret;
 }