diff options
author | Amerigo Wang <amwang@redhat.com> | 2012-10-28 13:43:53 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2012-11-03 14:50:15 -0400 |
commit | 1a9408355e91c21eaf4626386d65988a0ad7dc21 (patch) | |
tree | 17868f2436cf49159a58766b34bd77994990c377 | |
parent | 398f382c0a5bd62f0f31871e844700e1b9fd1ad3 (diff) |
ipv6: remove a useless NULL check
In dev_forward_change(), it is useless to check if idev->dev
is NULL, it is always non-NULL here.
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Cong Wang <amwang@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/ipv6/addrconf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv6/addrconf.c b/net/ipv6/addrconf.c index 387b813f227d..ced58e1d91b6 100644 --- a/net/ipv6/addrconf.c +++ b/net/ipv6/addrconf.c | |||
@@ -607,7 +607,7 @@ static void dev_forward_change(struct inet6_dev *idev) | |||
607 | dev = idev->dev; | 607 | dev = idev->dev; |
608 | if (idev->cnf.forwarding) | 608 | if (idev->cnf.forwarding) |
609 | dev_disable_lro(dev); | 609 | dev_disable_lro(dev); |
610 | if (dev && (dev->flags & IFF_MULTICAST)) { | 610 | if (dev->flags & IFF_MULTICAST) { |
611 | if (idev->cnf.forwarding) | 611 | if (idev->cnf.forwarding) |
612 | ipv6_dev_mc_inc(dev, &in6addr_linklocal_allrouters); | 612 | ipv6_dev_mc_inc(dev, &in6addr_linklocal_allrouters); |
613 | else | 613 | else |