]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
driver core: Delete an unnecessary check before the function call "put_device"
authorMarkus Elfring <elfring@users.sourceforge.net>
Thu, 5 Feb 2015 10:48:26 +0000 (11:48 +0100)
committerJiri Slaby <jslaby@suse.cz>
Thu, 15 Dec 2016 08:19:38 +0000 (09:19 +0100)
commit 5f0163a5ee9cc7c59751768bdfd94a73186debba upstream.

The put_device() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.

This issue was detected by using the Coccinelle software.

Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
drivers/base/core.c

index 944fecd32e9fe97816b5fce7dd67ee0d5abadfab..90458b1719a89edbaee1a28d6a506de3e961fb0b 100644 (file)
@@ -1185,8 +1185,7 @@ done:
        kobject_del(&dev->kobj);
  Error:
        cleanup_device_parent(dev);
-       if (parent)
-               put_device(parent);
+       put_device(parent);
 name_error:
        kfree(dev->p);
        dev->p = NULL;