dev_hold() already checks if its argument is NULL. Signed-off-by: Thorsten Blum --- net/rose/rose_route.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/net/rose/rose_route.c b/net/rose/rose_route.c index b72bf8a08d48..35e21a2bec9c 100644 --- a/net/rose/rose_route.c +++ b/net/rose/rose_route.c @@ -608,8 +608,7 @@ struct net_device *rose_dev_first(void) if (first == NULL || strncmp(dev->name, first->name, 3) < 0) first = dev; } - if (first) - dev_hold(first); + dev_hold(first); rcu_read_unlock(); return first; -- 2.50.0