net: pktgen: Deletion of an unnecessary check before the function call "proc_remove"
authorMarkus Elfring <elfring@users.sourceforge.net>
Tue, 18 Nov 2014 19:10:34 +0000 (20:10 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Nov 2014 20:20:15 +0000 (15:20 -0500)
The proc_remove() 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/core/pktgen.c

index 443256bdcddc8e01c2ba0f0b01c0b0bf78a6608b..da934fc3faa824458b6c8a139e929d72876d57d2 100644 (file)
@@ -3728,8 +3728,7 @@ static int pktgen_remove_device(struct pktgen_thread *t,
        /* Remove proc before if_list entry, because add_device uses
         * list to determine if interface already exist, avoid race
         * with proc_create_data() */
-       if (pkt_dev->entry)
-               proc_remove(pkt_dev->entry);
+       proc_remove(pkt_dev->entry);
 
        /* And update the thread if_list */
        _rem_dev_from_if_list(t, pkt_dev);