diff options
author | Robert Watson <rwatson@FreeBSD.org> | 2009-06-22 10:59:34 +0000 |
---|---|---|
committer | Robert Watson <rwatson@FreeBSD.org> | 2009-06-22 10:59:34 +0000 |
commit | 8896f83a58b5c4ff6288c8f101df25e92a371d83 (patch) | |
tree | 45a4f946feeb75c5ed5e8dcc028809c6966e1ba1 | |
parent | f863f9cbd9abad9ed571176939c6fb48f861d3e2 (diff) | |
download | src-8896f83a58b5c4ff6288c8f101df25e92a371d83.tar.gz src-8896f83a58b5c4ff6288c8f101df25e92a371d83.zip |
Add a new function, ifa_ifwithaddr_check(), which rather than returning
a pointer to an ifaddr matching the passed socket address, returns a
boolean indicating whether one was present. In the (near) future,
ifa_ifwithaddr() will return a referenced ifaddr rather than a raw
ifaddr pointer, and the new wrapper will allow callers that care only
about the boolean condition to avoid having to free that reference.
MFC after: 3 weeks
Notes
Notes:
svn path=/head/; revision=194622
-rw-r--r-- | sys/dev/cxgb/ulp/iw_cxgb/iw_cxgb_cm.c | 7 | ||||
-rw-r--r-- | sys/net/if.c | 18 | ||||
-rw-r--r-- | sys/net/if_var.h | 1 | ||||
-rw-r--r-- | sys/net/route.c | 2 | ||||
-rw-r--r-- | sys/netinet/in_pcb.c | 2 | ||||
-rw-r--r-- | sys/netinet/raw_ip.c | 2 | ||||
-rw-r--r-- | sys/netipx/ipx_pcb.c | 2 |
7 files changed, 25 insertions, 9 deletions
diff --git a/sys/dev/cxgb/ulp/iw_cxgb/iw_cxgb_cm.c b/sys/dev/cxgb/ulp/iw_cxgb/iw_cxgb_cm.c index 83b1b0de0137..c582dbda39cc 100644 --- a/sys/dev/cxgb/ulp/iw_cxgb/iw_cxgb_cm.c +++ b/sys/dev/cxgb/ulp/iw_cxgb/iw_cxgb_cm.c @@ -1337,12 +1337,13 @@ static int is_loopback_dst(struct iw_cm_id *cm_id) { uint16_t port = cm_id->remote_addr.sin_port; - struct ifaddr *ifa; + int ifa_present; cm_id->remote_addr.sin_port = 0; - ifa = ifa_ifwithaddr((struct sockaddr *)&cm_id->remote_addr); + ifa_present = ifa_ifwithaddr_check( + (struct sockaddr *)&cm_id->remote_addr); cm_id->remote_addr.sin_port = port; - return (ifa != NULL); + return (ifa_present); } int diff --git a/sys/net/if.c b/sys/net/if.c index bc135014cb20..eb5e519b2b2b 100644 --- a/sys/net/if.c +++ b/sys/net/if.c @@ -1466,8 +1466,8 @@ ifa_free(struct ifaddr *ifa) * Locate an interface based on a complete address. */ /*ARGSUSED*/ -struct ifaddr * -ifa_ifwithaddr(struct sockaddr *addr) +static struct ifaddr * +ifa_ifwithaddr_internal(struct sockaddr *addr) { INIT_VNET_NET(curvnet); struct ifnet *ifp; @@ -1500,6 +1500,20 @@ done: return (ifa); } +struct ifaddr * +ifa_ifwithaddr(struct sockaddr *addr) +{ + + return (ifa_ifwithaddr_internal(addr)); +} + +int +ifa_ifwithaddr_check(struct sockaddr *addr) +{ + + return (ifa_ifwithaddr_internal(addr) != NULL); +} + /* * Locate an interface based on the broadcast address. */ diff --git a/sys/net/if_var.h b/sys/net/if_var.h index cbd274afc9fb..35e4232b0a50 100644 --- a/sys/net/if_var.h +++ b/sys/net/if_var.h @@ -820,6 +820,7 @@ void ifq_init(struct ifaltq *, struct ifnet *ifp); void ifq_delete(struct ifaltq *); struct ifaddr *ifa_ifwithaddr(struct sockaddr *); +int ifa_ifwithaddr_check(struct sockaddr *); struct ifaddr *ifa_ifwithbroadaddr(struct sockaddr *); struct ifaddr *ifa_ifwithdstaddr(struct sockaddr *); struct ifaddr *ifa_ifwithnet(struct sockaddr *); diff --git a/sys/net/route.c b/sys/net/route.c index 1705e98e0193..2ebdeaa24108 100644 --- a/sys/net/route.c +++ b/sys/net/route.c @@ -547,7 +547,7 @@ rtredirect_fib(struct sockaddr *dst, if (!(flags & RTF_DONE) && rt && (!sa_equal(src, rt->rt_gateway) || rt->rt_ifa != ifa)) error = EINVAL; - else if (ifa_ifwithaddr(gateway)) + else if (ifa_ifwithaddr_check(gateway)) error = EHOSTUNREACH; if (error) goto done; diff --git a/sys/netinet/in_pcb.c b/sys/netinet/in_pcb.c index 252c7fb7d5e0..4e30572b88b7 100644 --- a/sys/netinet/in_pcb.c +++ b/sys/netinet/in_pcb.c @@ -357,7 +357,7 @@ in_pcbbind_setup(struct inpcb *inp, struct sockaddr *nam, in_addr_t *laddrp, * to any endpoint address, local or not. */ if ((inp->inp_flags & INP_BINDANY) == 0 && - ifa_ifwithaddr((struct sockaddr *)sin) == NULL) + ifa_ifwithaddr_check((struct sockaddr *)sin) == 0) return (EADDRNOTAVAIL); } laddr = sin->sin_addr; diff --git a/sys/netinet/raw_ip.c b/sys/netinet/raw_ip.c index 377e56b1c5d3..0157afbe250b 100644 --- a/sys/netinet/raw_ip.c +++ b/sys/netinet/raw_ip.c @@ -875,7 +875,7 @@ rip_bind(struct socket *so, struct sockaddr *nam, struct thread *td) (addr->sin_family != AF_INET && addr->sin_family != AF_IMPLINK) || (addr->sin_addr.s_addr && (inp->inp_flags & INP_BINDANY) == 0 && - ifa_ifwithaddr((struct sockaddr *)addr) == NULL)) + ifa_ifwithaddr_check((struct sockaddr *)addr) == 0)) return (EADDRNOTAVAIL); INP_INFO_WLOCK(&V_ripcbinfo); diff --git a/sys/netipx/ipx_pcb.c b/sys/netipx/ipx_pcb.c index bfe68c33849c..4c8eba0ae369 100644 --- a/sys/netipx/ipx_pcb.c +++ b/sys/netipx/ipx_pcb.c @@ -121,7 +121,7 @@ ipx_pcbbind(struct ipxpcb *ipxp, struct sockaddr *nam, struct thread *td) int tport = sipx->sipx_port; sipx->sipx_port = 0; /* yech... */ - if (ifa_ifwithaddr((struct sockaddr *)sipx) == NULL) + if (ifa_ifwithaddr_check((struct sockaddr *)sipx) == 0) return (EADDRNOTAVAIL); sipx->sipx_port = tport; } |