diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-02-05 05:48:26 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2015-03-25 08:52:35 -0400 |
commit | 5f0163a5ee9cc7c59751768bdfd94a73186debba (patch) | |
tree | 83c4790ac312bb3857d5a96930e556f093f5cf53 /drivers/base | |
parent | 7ca7ec40f42a13c6966fb3a5d8713e895d58c728 (diff) |
driver core: Delete an unnecessary check before the function call "put_device"
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>
Diffstat (limited to 'drivers/base')
-rw-r--r-- | drivers/base/core.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/base/core.c b/drivers/base/core.c index 07304a3b9ee2..e0998b6b6c49 100644 --- a/drivers/base/core.c +++ b/drivers/base/core.c | |||
@@ -1095,8 +1095,7 @@ done: | |||
1095 | kobject_del(&dev->kobj); | 1095 | kobject_del(&dev->kobj); |
1096 | Error: | 1096 | Error: |
1097 | cleanup_device_parent(dev); | 1097 | cleanup_device_parent(dev); |
1098 | if (parent) | 1098 | put_device(parent); |
1099 | put_device(parent); | ||
1100 | name_error: | 1099 | name_error: |
1101 | kfree(dev->p); | 1100 | kfree(dev->p); |
1102 | dev->p = NULL; | 1101 | dev->p = NULL; |