netlink: Deletion of an unnecessary check before the function call "__module_get"
authorMarkus Elfring <elfring@users.sourceforge.net>
Tue, 18 Nov 2014 20:03:13 +0000 (21:03 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Nov 2014 20:27:40 +0000 (15:27 -0500)
The __module_get() 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: David S. Miller <davem@davemloft.net>
net/netlink/af_netlink.c

index d479b32d5826aca27c66b4b59734059958fb67e5..e1aad6eeac147c04231e0762e91675414c02e4b5 100644 (file)
@@ -142,8 +142,7 @@ int netlink_add_tap(struct netlink_tap *nt)
        list_add_rcu(&nt->list, &netlink_tap_all);
        spin_unlock(&netlink_tap_lock);
 
-       if (nt->module)
-               __module_get(nt->module);
+       __module_get(nt->module);
 
        return 0;
 }