summaryrefslogtreecommitdiff
path: root/sys
diff options
context:
space:
mode:
authorozaki-r <ozaki-r@NetBSD.org>2016-06-22 07:48:17 +0000
committerozaki-r <ozaki-r@NetBSD.org>2016-06-22 07:48:17 +0000
commit720b6df78df07b31e059e12e50e3e07b79aa9e7c (patch)
tree2991ab643fb06afa1b04e6ca8ac146ad8a064d17 /sys
parent8599ef9c48459e3533bc1a57c931ef6f8fc8da08 (diff)
Remove unnecessary NULL checks of ifa->ifa_addr
If it's NULL, it should be a bug. There many IFADDR_FOREACH that don't do NULL check. If it can be NULL, they should fire already.
Diffstat (limited to 'sys')
-rw-r--r--sys/dist/pf/net/pf_if.c6
-rw-r--r--sys/net/if_stf.c6
-rw-r--r--sys/netinet6/in6.c10
-rw-r--r--sys/netinet6/in6_var.h4
-rw-r--r--sys/netinet6/ip6_input.c6
-rw-r--r--sys/netinet6/mld6.c8
6 files changed, 11 insertions, 29 deletions
diff --git a/sys/dist/pf/net/pf_if.c b/sys/dist/pf/net/pf_if.c
index 3ded67d61f3..8173f3d554c 100644
--- a/sys/dist/pf/net/pf_if.c
+++ b/sys/dist/pf/net/pf_if.c
@@ -1,4 +1,4 @@
-/* $NetBSD: pf_if.c,v 1.28 2016/06/16 02:38:40 ozaki-r Exp $ */
+/* $NetBSD: pf_if.c,v 1.29 2016/06/22 07:48:17 ozaki-r Exp $ */
/* $OpenBSD: pf_if.c,v 1.47 2007/07/13 09:17:48 markus Exp $ */
/*
@@ -34,7 +34,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: pf_if.c,v 1.28 2016/06/16 02:38:40 ozaki-r Exp $");
+__KERNEL_RCSID(0, "$NetBSD: pf_if.c,v 1.29 2016/06/22 07:48:17 ozaki-r Exp $");
#ifdef _KERNEL_OPT
#include "opt_inet.h"
@@ -560,8 +560,6 @@ pfi_instance_add(struct ifnet *ifp, int net, int flags)
if (ifp == NULL)
return;
IFADDR_FOREACH(ia, ifp) {
- if (ia->ifa_addr == NULL)
- continue;
af = ia->ifa_addr->sa_family;
if (af != AF_INET && af != AF_INET6)
continue;
diff --git a/sys/net/if_stf.c b/sys/net/if_stf.c
index 25c76f45c17..d57783c6485 100644
--- a/sys/net/if_stf.c
+++ b/sys/net/if_stf.c
@@ -1,4 +1,4 @@
-/* $NetBSD: if_stf.c,v 1.90 2016/06/10 13:31:44 ozaki-r Exp $ */
+/* $NetBSD: if_stf.c,v 1.91 2016/06/22 07:48:17 ozaki-r Exp $ */
/* $KAME: if_stf.c,v 1.62 2001/06/07 22:32:16 itojun Exp $ */
/*
@@ -75,7 +75,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_stf.c,v 1.90 2016/06/10 13:31:44 ozaki-r Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_stf.c,v 1.91 2016/06/22 07:48:17 ozaki-r Exp $");
#ifdef _KERNEL_OPT
#include "opt_inet.h"
@@ -304,8 +304,6 @@ stf_getsrcifa6(struct ifnet *ifp)
IFADDR_FOREACH(ifa, ifp)
{
- if (ifa->ifa_addr == NULL)
- continue;
if (ifa->ifa_addr->sa_family != AF_INET6)
continue;
sin6 = (struct sockaddr_in6 *)ifa->ifa_addr;
diff --git a/sys/netinet6/in6.c b/sys/netinet6/in6.c
index aa6b1e58da6..84dd49e3288 100644
--- a/sys/netinet6/in6.c
+++ b/sys/netinet6/in6.c
@@ -1,4 +1,4 @@
-/* $NetBSD: in6.c,v 1.199 2016/05/12 02:24:17 ozaki-r Exp $ */
+/* $NetBSD: in6.c,v 1.200 2016/06/22 07:48:17 ozaki-r Exp $ */
/* $KAME: in6.c,v 1.198 2001/07/18 09:12:38 itojun Exp $ */
/*
@@ -62,7 +62,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: in6.c,v 1.199 2016/05/12 02:24:17 ozaki-r Exp $");
+__KERNEL_RCSID(0, "$NetBSD: in6.c,v 1.200 2016/06/22 07:48:17 ozaki-r Exp $");
#ifdef _KERNEL_OPT
#include "opt_inet.h"
@@ -1688,8 +1688,6 @@ in6_ifinit(struct ifnet *ifp, struct in6_ifaddr *ia,
* and to validate the address if necessary.
*/
IFADDR_FOREACH(ifa, ifp) {
- if (ifa->ifa_addr == NULL)
- continue; /* just for safety */
if (ifa->ifa_addr->sa_family != AF_INET6)
continue;
ifacount++;
@@ -1758,8 +1756,6 @@ in6ifa_ifpforlinklocal(const struct ifnet *ifp, const int ignoreflags)
struct ifaddr *best_ifa = NULL, *ifa;
IFADDR_FOREACH(ifa, ifp) {
- if (ifa->ifa_addr == NULL)
- continue; /* just for safety */
if (ifa->ifa_addr->sa_family != AF_INET6)
continue;
if (!IN6_IS_ADDR_LINKLOCAL(IFA_IN6(ifa)))
@@ -1810,8 +1806,6 @@ in6ifa_ifpwithaddr(const struct ifnet *ifp, const struct in6_addr *addr)
struct ifaddr *best_ifa = NULL, *ifa;
IFADDR_FOREACH(ifa, ifp) {
- if (ifa->ifa_addr == NULL)
- continue; /* just for safety */
if (ifa->ifa_addr->sa_family != AF_INET6)
continue;
if (!IN6_ARE_ADDR_EQUAL(addr, IFA_IN6(ifa)))
diff --git a/sys/netinet6/in6_var.h b/sys/netinet6/in6_var.h
index 36dce75a5cd..91962b748bd 100644
--- a/sys/netinet6/in6_var.h
+++ b/sys/netinet6/in6_var.h
@@ -1,4 +1,4 @@
-/* $NetBSD: in6_var.h,v 1.76 2016/02/04 02:48:37 riastradh Exp $ */
+/* $NetBSD: in6_var.h,v 1.77 2016/06/22 07:48:17 ozaki-r Exp $ */
/* $KAME: in6_var.h,v 1.81 2002/06/08 11:16:51 itojun Exp $ */
/*
@@ -514,8 +514,6 @@ ifp_to_ia6(struct ifnet *ifp)
struct ifaddr *ifa;
IFADDR_FOREACH(ifa, ifp) {
- if (ifa->ifa_addr == NULL)
- continue;
if (ifa->ifa_addr->sa_family == AF_INET6)
break;
}
diff --git a/sys/netinet6/ip6_input.c b/sys/netinet6/ip6_input.c
index c98ace404a5..5b7d3b9bc77 100644
--- a/sys/netinet6/ip6_input.c
+++ b/sys/netinet6/ip6_input.c
@@ -1,4 +1,4 @@
-/* $NetBSD: ip6_input.c,v 1.160 2016/06/10 13:31:44 ozaki-r Exp $ */
+/* $NetBSD: ip6_input.c,v 1.161 2016/06/22 07:48:17 ozaki-r Exp $ */
/* $KAME: ip6_input.c,v 1.188 2001/03/29 05:34:31 itojun Exp $ */
/*
@@ -62,7 +62,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ip6_input.c,v 1.160 2016/06/10 13:31:44 ozaki-r Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ip6_input.c,v 1.161 2016/06/22 07:48:17 ozaki-r Exp $");
#ifdef _KERNEL_OPT
#include "opt_gateway.h"
@@ -534,8 +534,6 @@ ip6_input(struct mbuf *m, struct ifnet *rcvif)
*/
struct ifaddr *ifa;
IFADDR_FOREACH(ifa, rcvif) {
- if (ifa->ifa_addr == NULL)
- continue; /* just for safety */
if (ifa->ifa_addr->sa_family != AF_INET6)
continue;
if (IN6_ARE_ADDR_EQUAL(IFA_IN6(ifa), &ip6->ip6_dst)) {
diff --git a/sys/netinet6/mld6.c b/sys/netinet6/mld6.c
index 7588a9b5101..42bb6f0ee52 100644
--- a/sys/netinet6/mld6.c
+++ b/sys/netinet6/mld6.c
@@ -1,4 +1,4 @@
-/* $NetBSD: mld6.c,v 1.68 2016/06/21 03:28:27 ozaki-r Exp $ */
+/* $NetBSD: mld6.c,v 1.69 2016/06/22 07:48:17 ozaki-r Exp $ */
/* $KAME: mld6.c,v 1.25 2001/01/16 14:14:18 itojun Exp $ */
/*
@@ -102,7 +102,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: mld6.c,v 1.68 2016/06/21 03:28:27 ozaki-r Exp $");
+__KERNEL_RCSID(0, "$NetBSD: mld6.c,v 1.69 2016/06/22 07:48:17 ozaki-r Exp $");
#ifdef _KERNEL_OPT
#include "opt_inet.h"
@@ -998,8 +998,6 @@ in6_multicast_sysctl(SYSCTLFN_ARGS)
if (oldp == NULL) {
*oldlenp = 0;
IFADDR_FOREACH(ifa, ifp) {
- if (ifa->ifa_addr == NULL)
- continue;
if (ifa->ifa_addr->sa_family != AF_INET6)
continue;
ifa6 = (struct in6_ifaddr *)ifa;
@@ -1016,8 +1014,6 @@ in6_multicast_sysctl(SYSCTLFN_ARGS)
error = 0;
written = 0;
IFADDR_FOREACH(ifa, ifp) {
- if (ifa->ifa_addr == NULL)
- continue;
if (ifa->ifa_addr->sa_family != AF_INET6)
continue;
ifa6 = (struct in6_ifaddr *)ifa;