misc: mic: Release reference count and memory for VOP device
authorArvind Yadav <arvind.yadav.cs@gmail.com>
Mon, 12 Mar 2018 11:36:52 +0000 (17:06 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 15 Mar 2018 17:12:01 +0000 (18:12 +0100)
Never directly free @dev after calling device_register(),
even if it returned an error! Always use put_device() to
give up the reference initialized.
Release allocated memory for vop device in vop_release_dev().

Signed-off-by: Arvind Yadav <arvind.yadav.cs@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/misc/mic/bus/vop_bus.c

index fd7f2a6..e5bb9c7 100644 (file)
@@ -135,7 +135,9 @@ EXPORT_SYMBOL_GPL(vop_unregister_driver);
 
 static void vop_release_dev(struct device *d)
 {
-       put_device(d);
+       struct vop_device *dev = dev_to_vop(d);
+
+       kfree(dev);
 }
 
 struct vop_device *
@@ -174,7 +176,7 @@ vop_register_device(struct device *pdev, int id,
                goto free_vdev;
        return vdev;
 free_vdev:
-       kfree(vdev);
+       put_device(&vdev->dev);
        return ERR_PTR(ret);
 }
 EXPORT_SYMBOL_GPL(vop_register_device);