diff options
author | Jayachandran C <jchandra@digeo.com> | 2005-10-27 18:10:01 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@mandriva.com> | 2005-10-29 00:53:39 -0400 |
commit | 9fcc2e8a752f7d3d889114221b67c459557823e9 (patch) | |
tree | d06f567aabf247fd379a66177de5163299bc81d7 /net | |
parent | 360ac8e2f1a38c3497739636c3b702352d1ad0ae (diff) |
[IPV4]: Fix issue reported by Coverity in ipv4/fib_frontend.c
fib_del_ifaddr() dereferences ifa->ifa_dev, so the code already assumes that
ifa->ifa_dev is non-NULL, the check is unnecessary.
Signed-off-by: Jayachandran C. <c.jayachandran at gmail.com>
Signed-off-by: Arnaldo Carvalho de Melo <acme@mandriva.com>
Diffstat (limited to 'net')
-rw-r--r-- | net/ipv4/fib_frontend.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/ipv4/fib_frontend.c b/net/ipv4/fib_frontend.c index e61bc7177eb1..990633c09dfe 100644 --- a/net/ipv4/fib_frontend.c +++ b/net/ipv4/fib_frontend.c | |||
@@ -591,7 +591,7 @@ static int fib_inetaddr_event(struct notifier_block *this, unsigned long event, | |||
591 | break; | 591 | break; |
592 | case NETDEV_DOWN: | 592 | case NETDEV_DOWN: |
593 | fib_del_ifaddr(ifa); | 593 | fib_del_ifaddr(ifa); |
594 | if (ifa->ifa_dev && ifa->ifa_dev->ifa_list == NULL) { | 594 | if (ifa->ifa_dev->ifa_list == NULL) { |
595 | /* Last address was deleted from this interface. | 595 | /* Last address was deleted from this interface. |
596 | Disable IP. | 596 | Disable IP. |
597 | */ | 597 | */ |