RDMA/ipoib: Remove NULL check before dev_{put, hold}
authorJules Irenge <jbi.octave@gmail.com>
Tue, 30 Apr 2024 23:47:33 +0000 (00:47 +0100)
committerLeon Romanovsky <leon@kernel.org>
Thu, 2 May 2024 14:46:38 +0000 (17:46 +0300)
Coccinelle reports a warning

WARNING: NULL check before dev_{put, hold} functions is not needed

The reason is the call netdev_{put, hold} of dev_{put,hold} will check NULL
There is no need to check before using dev_{put, hold}

Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
Link: https://lore.kernel.org/r/ZjGDFatHRMI6Eg7M@octinomon.home
Signed-off-by: Leon Romanovsky <leon@kernel.org>
drivers/infiniband/ulp/ipoib/ipoib_main.c

index 6f2a688fccbfb02ae7bdf3d55cca0e77fa9b56b4..4abec0124ea37d19a804a8b0f16a5499f962bc0b 100644 (file)
@@ -329,8 +329,7 @@ static struct net_device *ipoib_get_master_net_dev(struct net_device *dev)
 
        rcu_read_lock();
        master = netdev_master_upper_dev_get_rcu(dev);
-       if (master)
-               dev_hold(master);
+       dev_hold(master);
        rcu_read_unlock();
 
        if (master)