diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-21 10:25:45 +0100 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2014-12-02 06:35:21 +0100 |
commit | 2ed37f6c9b69092afbbf2a990336f8cbd303947d (patch) | |
tree | ea8fef3d2bf536989ae391b7e5365600b227f74b /drivers/target/iscsi | |
parent | target: Fix target_core_register_fabric() for built-in fabric modules (diff) | |
download | linux-2ed37f6c9b69092afbbf2a990336f8cbd303947d.tar.xz linux-2ed37f6c9b69092afbbf2a990336f8cbd303947d.zip |
target: Deletion of unnecessary checks before the function call "module_put"
The module_put() 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: Nicholas Bellinger <nab@linux-iscsi.org>
Diffstat (limited to 'drivers/target/iscsi')
-rw-r--r-- | drivers/target/iscsi/iscsi_target_transport.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/target/iscsi/iscsi_target_transport.c b/drivers/target/iscsi/iscsi_target_transport.c index 882728fac30c..08217d62fb0d 100644 --- a/drivers/target/iscsi/iscsi_target_transport.c +++ b/drivers/target/iscsi/iscsi_target_transport.c @@ -26,8 +26,7 @@ struct iscsit_transport *iscsit_get_transport(int type) void iscsit_put_transport(struct iscsit_transport *t) { - if (t->owner) - module_put(t->owner); + module_put(t->owner); } int iscsit_register_transport(struct iscsit_transport *t) |