]> git.saurik.com Git - apple/xnu.git/blobdiff - bsd/netinet6/icmp6.c
xnu-4903.241.1.tar.gz
[apple/xnu.git] / bsd / netinet6 / icmp6.c
index cdf92a564c9939b260a1b8d21a31c2300266be4d..58917f864b49d49917d690d96059aeb6c33d8ce4 100644 (file)
@@ -1,8 +1,8 @@
 /*
- * Copyright (c) 2000-2012 Apple Inc. All rights reserved.
+ * Copyright (c) 2000-2018 Apple Inc. All rights reserved.
  *
  * @APPLE_OSREFERENCE_LICENSE_HEADER_START@
- * 
+ *
  * This file contains Original Code and/or Modifications of Original Code
  * as defined in and that are subject to the Apple Public Source License
  * Version 2.0 (the 'License'). You may not use this file except in
  * unlawful or unlicensed copies of an Apple operating system, or to
  * circumvent, violate, or enable the circumvention or violation of, any
  * terms of an Apple operating system software license agreement.
- * 
+ *
  * Please obtain a copy of the License at
  * http://www.opensource.apple.com/apsl/ and read it before using this file.
- * 
+ *
  * The Original Code and all software distributed under the License are
  * distributed on an 'AS IS' basis, WITHOUT WARRANTY OF ANY KIND, EITHER
  * EXPRESS OR IMPLIED, AND APPLE HEREBY DISCLAIMS ALL SUCH WARRANTIES,
@@ -22,7 +22,7 @@
  * FITNESS FOR A PARTICULAR PURPOSE, QUIET ENJOYMENT OR NON-INFRINGEMENT.
  * Please see the License for the specific language governing rights and
  * limitations under the License.
- * 
+ *
  * @APPLE_OSREFERENCE_LICENSE_HEADER_END@
  */
 
 #include <netinet6/mld6_var.h>
 #include <netinet/in_pcb.h>
 #include <netinet6/in6_pcb.h>
+#include <netinet6/in6_var.h>
 #include <netinet6/nd6.h>
 #include <netinet6/in6_ifattach.h>
 #include <netinet6/ip6protosw.h>
 #if IPSEC
 #include <netinet6/ipsec.h>
 #include <netkey/key.h>
-
-extern int ipsec_bypass;
 #endif
 
 #include <net/net_osdep.h>
 
-extern struct domain inet6domain;
-extern struct ip6protosw inet6sw[];
+#if NECP
+#include <net/necp.h>
+#endif
+
 extern struct ip6protosw *ip6_protox[];
 
 extern uint32_t rip_sendspace;
@@ -147,24 +148,23 @@ struct icmp6stat icmp6stat;
 
 extern struct inpcbhead ripcb;
 extern int icmp6errppslim;
+extern int icmp6rappslim;
 static int icmp6errpps_count = 0;
+static int icmp6rapps_count = 0;
 static struct timeval icmp6errppslim_last;
+static struct timeval icmp6rappslim_last;
 extern int icmp6_nodeinfo;
 extern struct inpcbinfo ripcbinfo;
-extern lck_mtx_t *inet6_domain_mutex;
 
 static void icmp6_errcount(struct icmp6errstat *, int, int);
 static int icmp6_rip6_input(struct mbuf **, int);
 static int icmp6_ratelimit(const struct in6_addr *, const int, const int);
 static const char *icmp6_redirect_diag(struct in6_addr *,
        struct in6_addr *, struct in6_addr *);
-#ifndef HAVE_PPSRATECHECK
-static int ppsratecheck(struct timeval *, int *, int);
-#endif
 static struct mbuf *ni6_input(struct mbuf *, int);
 static struct mbuf *ni6_nametodns(const char *, int, int);
 static int ni6_dnsmatch(const char *, int, const char *, int);
-static int ni6_addrs(struct icmp6_nodeinfo *, 
+static int ni6_addrs(struct icmp6_nodeinfo *,
                          struct ifnet **, char *);
 static int ni6_store_addrs(struct icmp6_nodeinfo *, struct icmp6_nodeinfo *,
                                struct ifnet *, int);
@@ -173,15 +173,24 @@ static int icmp6_notify_error(struct mbuf *, int, int, int);
 
 
 void
-icmp6_init()
+icmp6_init(struct ip6protosw *pp, struct domain *dp)
 {
-       mld_init();
+#pragma unused(dp)
+       static int icmp6_initialized = 0;
+
+       /* Also called from ip6_init() without pp */
+       VERIFY(pp == NULL ||
+           (pp->pr_flags & (PR_INITIALIZED|PR_ATTACHED)) == PR_ATTACHED);
+
+       /* This gets called by more than one protocols, so initialize once */
+       if (!icmp6_initialized) {
+               icmp6_initialized = 1;
+               mld_init();
+       }
 }
 
 static void
-icmp6_errcount(stat, type, code)
-       struct icmp6errstat *stat;
-       int type, code;
+icmp6_errcount(struct icmp6errstat *stat, int type, int code)
 {
        switch (type) {
        case ICMP6_DST_UNREACH:
@@ -273,7 +282,11 @@ icmp6_error2(struct mbuf *m, int type, int code, int param,
  * Generate an error packet of type error in response to bad IP6 packet.
  */
 void
-icmp6_error(struct mbuf *m, int type, int code, int param)
+icmp6_error(struct mbuf *m, int type, int code, int param) {
+       icmp6_error_flag(m, type, code, param, ICMP6_ERROR_RST_MRCVIF);
+}
+
+void icmp6_error_flag (struct mbuf *m, int type, int code, int param, int flags)
 {
        struct ip6_hdr *oip6, *nip6;
        struct icmp6_hdr *icmp6;
@@ -382,7 +395,7 @@ icmp6_error(struct mbuf *m, int type, int code, int param)
                m_adj(m, ICMPV6_PLD_MAXLEN - m->m_pkthdr.len);
 
        preplen = sizeof(struct ip6_hdr) + sizeof(struct icmp6_hdr);
-       M_PREPEND(m, preplen, M_DONTWAIT);
+       M_PREPEND(m, preplen, M_DONTWAIT, 1);
        if (m && m->m_len < preplen)
                m = m_pullup(m, preplen);
        if (m == NULL) {
@@ -409,7 +422,9 @@ icmp6_error(struct mbuf *m, int type, int code, int param)
         * clear m->m_pkthdr.rcvif for safety, we should have enough scope
         * information in ip header (nip6).
         */
-       m->m_pkthdr.rcvif = NULL;
+       if (flags & ICMP6_ERROR_RST_MRCVIF) {
+               m->m_pkthdr.rcvif = NULL;
+       }
 
        icmp6stat.icp6s_outhist[type]++;
        icmp6_reflect(m, sizeof(struct ip6_hdr)); /* header order: IPv6 - ICMPv6 */
@@ -458,6 +473,42 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                goto freeit;
        }
 
+#ifndef PULLDOWN_TEST
+       icmp6 = (struct icmp6_hdr *)((caddr_t)ip6 + off);
+#else
+       IP6_EXTHDR_GET(icmp6, struct icmp6_hdr *, m, off, sizeof(*icmp6));
+       if (icmp6 == NULL) {
+               icmp6stat.icp6s_tooshort++;
+               return IPPROTO_DONE;
+       }
+#endif
+       code = icmp6->icmp6_code;
+
+       /*
+        * Early check for RFC 6980
+        * Drop certain NDP packets if they came in fragmented
+        */
+       switch (icmp6->icmp6_type) {
+       case ND_ROUTER_SOLICIT:
+       case ND_ROUTER_ADVERT:
+       case ND_NEIGHBOR_SOLICIT:
+       case ND_NEIGHBOR_ADVERT:
+       case ND_REDIRECT:
+               if (m->m_pkthdr.pkt_flags & PKTF_REASSEMBLED) {
+                       icmp6stat.icp6s_rfc6980_drop++;
+                       goto freeit;
+               }
+               break;
+       default:
+               break;
+       }
+
+       /* Apply rate limit before checksum validation. */
+       if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
+               icmp6stat.icp6s_toofreq++;
+               goto freeit;
+       }
+
        /*
         * Check multicast group membership.
         * Note: SSM filters are not applied for ICMPv6 traffic.
@@ -474,7 +525,7 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                         * Don't discard if this is a Neighbor Solicitation
                         * that needs to be proxied (see check down below.)
                         */
-                       if (!(m->m_pkthdr.aux_flags & MAUXF_PROXY_DST)) {
+                       if (!(m->m_pkthdr.pkt_flags & PKTF_PROXY_DST)) {
                                ip6stat.ip6s_notmember++;
                                in6_ifstat_inc(m->m_pkthdr.rcvif,
                                    ifs6_in_discard);
@@ -488,17 +539,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
        /*
         * calculate the checksum
         */
-#ifndef PULLDOWN_TEST
-       icmp6 = (struct icmp6_hdr *)((caddr_t)ip6 + off);
-#else
-       IP6_EXTHDR_GET(icmp6, struct icmp6_hdr *, m, off, sizeof(*icmp6));
-       if (icmp6 == NULL) {
-               icmp6stat.icp6s_tooshort++;
-               return IPPROTO_DONE;
-       }
-#endif
-       code = icmp6->icmp6_code;
-
        if ((sum = in6_cksum(m, IPPROTO_ICMPV6, off, icmp6len)) != 0) {
                nd6log((LOG_ERR,
                    "ICMP6 checksum error(%d|%x) %s\n",
@@ -507,7 +547,7 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                goto freeit;
        }
 
-       if (m->m_pkthdr.aux_flags & MAUXF_PROXY_DST) {
+       if (m->m_pkthdr.pkt_flags & PKTF_PROXY_DST) {
                /*
                 * This is the special case of proxying NS (dst is either
                 * solicited-node multicast or unicast); process it locally
@@ -534,15 +574,13 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_dstunreach);
                switch (code) {
                case ICMP6_DST_UNREACH_NOROUTE:
+               case ICMP6_DST_UNREACH_ADDR:    /* PRC_HOSTDEAD is a DOS */
                        code = PRC_UNREACH_NET;
                        break;
                case ICMP6_DST_UNREACH_ADMIN:
                        icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_adminprohib);
                        code = PRC_UNREACH_PROTOCOL; /* is this a good code? */
                        break;
-               case ICMP6_DST_UNREACH_ADDR:
-                       code = PRC_HOSTDEAD;
-                       break;
                case ICMP6_DST_UNREACH_BEYONDSCOPE:
                        /* I mean "source address was incorrect." */
                        code = PRC_PARAMPROB;
@@ -554,7 +592,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                        goto badcode;
                }
                goto deliver;
-               break;
 
        case ICMP6_PACKET_TOO_BIG:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_pkttoobig);
@@ -566,7 +603,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                 * intermediate extension headers.
                 */
                goto deliver;
-               break;
 
        case ICMP6_TIME_EXCEEDED:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_timeexceed);
@@ -581,7 +617,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                        goto badcode;
                }
                goto deliver;
-               break;
 
        case ICMP6_PARAM_PROB:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_paramprob);
@@ -597,22 +632,15 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                        goto badcode;
                }
                goto deliver;
-               break;
 
        case ICMP6_ECHO_REQUEST:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_echo);
                if (code != 0)
                        goto badcode;
 
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
                if ((n = m_copy(m, 0, M_COPYALL)) == NULL) {
                        /* Give up remote */
                        goto rate_limit_checked;
-                       break;
                }
                if ((n->m_flags & M_EXT) != 0
                 || n->m_len < off + sizeof(struct icmp6_hdr)) {
@@ -627,7 +655,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                                /* Give up remote */
                                m_freem(n0);
                                goto rate_limit_checked;
-                               break;
                        }
                        MGETHDR(n, M_DONTWAIT, n0->m_type);     /* MAC-OK */
                        if (n && maxlen >= MHLEN) {
@@ -641,7 +668,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                                /* Give up remote */
                                m_freem(n0);
                                goto rate_limit_checked;
-                               break;
                        }
                        M_COPY_PKTHDR(n, n0);
                        /*
@@ -680,7 +706,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                        icmp6_reflect(n, noff);
                }
                goto rate_limit_checked;
-               break;
 
        case ICMP6_ECHO_REPLY:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_echoreply);
@@ -698,11 +723,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                else
                        icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_mldreport);
 
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
                if ((n = m_copym(m, 0, M_COPYALL, M_DONTWAIT)) == NULL) {
                        /* give up local */
                        if (mld_input(m, off, icmp6len) == IPPROTO_DONE)
@@ -713,7 +733,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                        m_freem(n);
                /* m stays. */
                goto rate_limit_checked;
-               break;
 
        case MLD_LISTENER_DONE:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_mlddone);
@@ -731,7 +750,7 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                if (!icmp6_nodeinfo)
                        break;
 //### LD 10/20 Check fbsd differences here. Not sure we're more advanced or not.
-               /* By RFC 4620 refuse to answer queries from global scope addresses */ 
+               /* By RFC 4620 refuse to answer queries from global scope addresses */
                if ((icmp6_nodeinfo & 8) != 8 && in6_addrscope(&ip6->ip6_src) == IPV6_ADDR_SCOPE_GLOBAL)
                        break;
 
@@ -742,10 +761,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                IP6_EXTHDR_CHECK(m, off, sizeof(struct icmp6_nodeinfo),
                                 return IPPROTO_DONE);
 #endif
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
 
                n = m_copy(m, 0, M_COPYALL);
                if (n)
@@ -757,7 +772,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                        icmp6_reflect(n, noff);
                }
                goto rate_limit_checked;
-               break;
 
        case ICMP6_WRUREPLY:
                if (code != 0)
@@ -771,11 +785,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                if (icmp6len < sizeof(struct nd_router_solicit))
                        goto badlen;
 
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
                if ((n = m_copym(m, 0, M_COPYALL, M_DONTWAIT)) == NULL) {
                        /* give up local */
                        nd6_rs_input(m, off, icmp6len);
@@ -785,7 +794,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                nd6_rs_input(n, off, icmp6len);
                /* m stays. */
                goto rate_limit_checked;
-               break;
 
        case ND_ROUTER_ADVERT:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_routeradvert);
@@ -794,11 +802,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                if (icmp6len < sizeof(struct nd_router_advert))
                        goto badlen;
 
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
                if ((n = m_copym(m, 0, M_COPYALL, M_DONTWAIT)) == NULL) {
                        /* give up local */
                        nd6_ra_input(m, off, icmp6len);
@@ -808,7 +811,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                nd6_ra_input(n, off, icmp6len);
                /* m stays. */
                goto rate_limit_checked;
-               break;
 
        case ND_NEIGHBOR_SOLICIT:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_neighborsolicit);
@@ -817,12 +819,8 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                if (icmp6len < sizeof(struct nd_neighbor_solicit))
                        goto badlen;
 
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
-               if ((n = m_copym(m, 0, M_COPYALL, M_DONTWAIT)) == NULL) {
+               if (proxy ||
+                   ((n = m_copym(m, 0, M_COPYALL, M_DONTWAIT)) == NULL)) {
                        /* give up local */
                        nd6_ns_input(m, off, icmp6len);
                        m = NULL;
@@ -831,7 +829,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                nd6_ns_input(n, off, icmp6len);
                /* m stays. */
                goto rate_limit_checked;
-               break;
 
        case ND_NEIGHBOR_ADVERT:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_neighboradvert);
@@ -840,11 +837,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                if (icmp6len < sizeof(struct nd_neighbor_advert))
                        goto badlen;
 
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
                if ((n = m_copym(m, 0, M_COPYALL, M_DONTWAIT)) == NULL) {
                        /* give up local */
                        nd6_na_input(m, off, icmp6len);
@@ -854,7 +846,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                nd6_na_input(n, off, icmp6len);
                /* m stays. */
                goto rate_limit_checked;
-               break;
 
        case ND_REDIRECT:
                icmp6_ifstat_inc(m->m_pkthdr.rcvif, ifs6_in_redirect);
@@ -863,11 +854,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                if (icmp6len < sizeof(struct nd_redirect))
                        goto badlen;
 
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
                if ((n = m_copym(m, 0, M_COPYALL, M_DONTWAIT)) == NULL) {
                        /* give up local */
                        icmp6_redirect_input(m, off);
@@ -877,7 +863,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                icmp6_redirect_input(n, off);
                /* m stays. */
                goto rate_limit_checked;
-               break;
 
        case ICMP6_ROUTER_RENUMBERING:
                if (code != ICMP6_ROUTER_RENUMBERING_COMMAND &&
@@ -888,11 +873,6 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                break;
 
        default:
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
                nd6log((LOG_DEBUG,
                    "icmp6_input: unknown type %d(src=%s, dst=%s, ifid=%d)\n",
                    icmp6->icmp6_type, ip6_sprintf(&ip6->ip6_src),
@@ -905,14 +885,8 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                } else {
                        /* ICMPv6 informational: MUST not deliver */
                        goto rate_limit_checked;
-                       break;
                }
        deliver:
-               if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-                       icmp6stat.icp6s_toofreq++;
-                       goto freeit;
-               }
-
                if (icmp6_notify_error(m, off, icmp6len, code)) {
                        /* In this case, m should've been freed. */
                        return(IPPROTO_DONE);
@@ -928,16 +902,9 @@ icmp6_input(struct mbuf **mp, int *offp, int proto)
                break;
        }
 
-       if (icmp6_ratelimit(&ip6->ip6_dst, icmp6->icmp6_type, code)) {
-               icmp6stat.icp6s_toofreq++;
-               goto freeit;
-       }
 rate_limit_checked:
-       /* deliver the packet to appropriate sockets (unless proxying) */
-       if (!proxy) {
-               icmp6_rip6_input(&m, *offp);
-               return IPPROTO_DONE;
-       }
+       icmp6_rip6_input(&m, *offp);
+       return IPPROTO_DONE;
 
 freeit:
        m_freem(m);
@@ -945,9 +912,7 @@ freeit:
 }
 
 static int
-icmp6_notify_error(m, off, icmp6len, code)
-       struct mbuf *m;
-       int off, icmp6len, code;
+icmp6_notify_error(struct mbuf *m, int off, int icmp6len, int code)
 {
        struct icmp6_hdr *icmp6;
        struct ip6_hdr *eip6;
@@ -975,7 +940,7 @@ icmp6_notify_error(m, off, icmp6len, code)
 
        /* Detect the upper level protocol */
        {
-               void (*ctlfunc)(int, struct sockaddr *, void *);
+               void (*ctlfunc)(int, struct sockaddr *, void *, struct ifnet *);
                u_int8_t nxt = eip6->ip6_nxt;
                int eoff = off + sizeof(struct icmp6_hdr) +
                        sizeof(struct ip6_hdr);
@@ -1166,11 +1131,16 @@ icmp6_notify_error(m, off, icmp6len, code)
                        icmp6_mtudisc_update(&ip6cp, 1);        /*XXX*/
                }
 
-               ctlfunc = (void (*)(int, struct sockaddr *, void *))
-                       (ip6_protox[nxt]->pr_ctlinput);
+               ctlfunc = ip6_protox[nxt]->pr_ctlinput;
                if (ctlfunc) {
+                       LCK_MTX_ASSERT(inet6_domain_mutex, LCK_MTX_ASSERT_OWNED);
+
+                       lck_mtx_unlock(inet6_domain_mutex);
+
                        (void) (*ctlfunc)(code, (struct sockaddr *)&icmp6dst,
-                                         &ip6cp);
+                           &ip6cp, m->m_pkthdr.rcvif);
+
+                       lck_mtx_lock(inet6_domain_mutex);
                }
        }
        return(0);
@@ -1181,9 +1151,7 @@ freeit:
 }
 
 void
-icmp6_mtudisc_update(ip6cp, validated)
-       struct ip6ctlparam *ip6cp;
-       int validated;
+icmp6_mtudisc_update(struct ip6ctlparam *ip6cp, int validated)
 {
        struct in6_addr *dst = ip6cp->ip6c_finaldst;
        struct icmp6_hdr *icmp6 = ip6cp->ip6c_icmp6;
@@ -1220,6 +1188,13 @@ icmp6_mtudisc_update(ip6cp, validated)
                    htons(m->m_pkthdr.rcvif->if_index);
        }
        /* sin6.sin6_scope_id = XXX: should be set if DST is a scoped addr */
+       /*
+        * XXX On a side note, for asymmetric data-path
+        * the lookup on receive interace is probably not
+        * what we want to do.
+        * That requires looking at the cached route for the
+        * protocol control block.
+        */
        rt = rtalloc1_scoped((struct sockaddr *)&sin6, 0,
            RTF_CLONING | RTF_PRCLONING, m->m_pkthdr.rcvif->if_index);
        if (rt != NULL) {
@@ -1249,9 +1224,7 @@ icmp6_mtudisc_update(ip6cp, validated)
  */
 #define hostnamelen    strlen(hostname)
 static struct mbuf *
-ni6_input(m, off)
-       struct mbuf *m;
-       int off;
+ni6_input(struct mbuf *m, int off)
 {
        struct icmp6_nodeinfo *ni6, *nni6;
        struct mbuf *n = NULL;
@@ -1309,23 +1282,18 @@ ni6_input(m, off)
                        goto bad;
                /* else it's a link-local multicast, fine */
        } else {                /* unicast or anycast */
-               struct in6_ifaddr *ia6;
+               uint32_t ia6_flags;
 
-               if ((ia6 = ip6_getdstifaddr(m)) == NULL)
+               if (ip6_getdstifaddr_info(m, NULL, &ia6_flags) != 0)
                        goto bad; /* XXX impossible */
 
-               IFA_LOCK(&ia6->ia_ifa);
-               if ((ia6->ia6_flags & IN6_IFF_TEMPORARY) &&
+               if ((ia6_flags & IN6_IFF_TEMPORARY) &&
                    !(icmp6_nodeinfo & ICMP6_NODEINFO_TMPADDROK)) {
                        nd6log((LOG_DEBUG, "ni6_input: ignore node info to "
                                "a temporary address in %s:%d",
-                              __FILE__, __LINE__));
-                       IFA_UNLOCK(&ia6->ia_ifa);
-                       IFA_REMREF(&ia6->ia_ifa);
+                              __func__, __LINE__));
                        goto bad;
                }
-               IFA_UNLOCK(&ia6->ia_ifa);
-               IFA_REMREF(&ia6->ia_ifa);
        }
 
        /* validate query Subject field. */
@@ -1595,10 +1563,10 @@ bad:
  * treated as truncated name (two \0 at the end).  this is a wild guess.
  */
 static struct mbuf *
-ni6_nametodns(name, namelen, old)
-       const char *name;
-       int namelen;
-       int old;        /* return pascal string if non-zero */
+ni6_nametodns(
+       const char *name,
+       int namelen,
+       int old)        /* return pascal string if non-zero */
 {
        struct mbuf *m;
        char *cp, *ep;
@@ -1695,11 +1663,7 @@ ni6_nametodns(name, namelen, old)
  * XXX upper/lowercase match (see RFC2065)
  */
 static int
-ni6_dnsmatch(a, alen, b, blen)
-       const char *a;
-       int alen;
-       const char *b;
-       int blen;
+ni6_dnsmatch(const char *a, int alen, const char *b, int blen)
 {
        const char *a0, *b0;
        int l;
@@ -1759,10 +1723,7 @@ ni6_dnsmatch(a, alen, b, blen)
  * calculate the number of addresses to be returned in the node info reply.
  */
 static int
-ni6_addrs(ni6, ifpp, subj)
-       struct icmp6_nodeinfo *ni6;
-       struct ifnet **ifpp;
-       char *subj;
+ni6_addrs(struct icmp6_nodeinfo *ni6, struct ifnet **ifpp, char *subj)
 {
        struct ifnet *ifp;
        struct in6_ifaddr *ifa6;
@@ -1878,10 +1839,8 @@ ni6_addrs(ni6, ifpp, subj)
 }
 
 static int
-ni6_store_addrs(ni6, nni6, ifp0, resid)
-       struct icmp6_nodeinfo *ni6, *nni6;
-       struct ifnet *ifp0;
-       int resid;
+ni6_store_addrs(struct icmp6_nodeinfo *ni6, struct icmp6_nodeinfo *nni6,
+               struct ifnet *ifp0, int resid)
 {
        struct ifnet *ifp = ifp0;
        struct in6_ifaddr *ifa6;
@@ -1891,9 +1850,7 @@ ni6_store_addrs(ni6, nni6, ifp0, resid)
        u_char *cp = (u_char *)(nni6 + 1);
        int niflags = ni6->ni_flags;
        u_int32_t ltime;
-       struct timeval timenow;
-
-       getmicrotime(&timenow);
+       uint64_t now = net_uptime();
 
        if (ifp0 == NULL && !(niflags & NI_NODEADDR_FLAG_ALL))
                return (0);     /* needless to copy */
@@ -1908,6 +1865,8 @@ ni6_store_addrs(ni6, nni6, ifp0, resid)
                ifnet_lock_shared(ifp);
                for (ifa = ifp->if_addrlist.tqh_first; ifa;
                     ifa = ifa->ifa_list.tqe_next) {
+                       struct in6_addrlifetime_i *lt;
+
                        IFA_LOCK(ifa);
                        if (ifa->ifa_addr->sa_family != AF_INET6) {
                                IFA_UNLOCK(ifa);
@@ -2003,13 +1962,12 @@ ni6_store_addrs(ni6, nni6, ifp0, resid)
                         * address configuration by DHCPv6, so the former
                         * case can't happen.
                         */
-                       if (ifa6->ia6_lifetime.ia6t_expire == 0 &&
-                           (ifa6->ia6_flags & IN6_IFF_TEMPORARY) == 0)
+                       lt = &ifa6->ia6_lifetime;
+                       if (lt->ia6ti_expire == 0) {
                                ltime = ND6_INFINITE_LIFETIME;
-                       else {
-                               if (ifa6->ia6_lifetime.ia6t_expire >
-                                   timenow.tv_sec)
-                                       ltime = htonl(ifa6->ia6_lifetime.ia6t_expire - timenow.tv_sec);
+                       } else {
+                               if (lt->ia6ti_expire > now)
+                                       ltime = htonl(lt->ia6ti_expire - now);
                                else
                                        ltime = 0;
                        }
@@ -2050,9 +2008,7 @@ ni6_store_addrs(ni6, nni6, ifp0, resid)
  * XXX almost dup'ed code with rip6_input.
  */
 static int
-icmp6_rip6_input(mp, off)
-       struct  mbuf **mp;
-       int     off;
+icmp6_rip6_input(struct mbuf **mp, int off)
 {
        struct mbuf *m = *mp;
        struct ip6_hdr *ip6 = mtod(m, struct ip6_hdr *);
@@ -2062,6 +2018,7 @@ icmp6_rip6_input(mp, off)
        struct icmp6_hdr *icmp6;
        struct mbuf *opts = NULL;
        int ret = 0;
+       struct ifnet *ifp = m->m_pkthdr.rcvif;
 
 #ifndef PULLDOWN_TEST
        /* this is assumed to be safe. */
@@ -2082,10 +2039,10 @@ icmp6_rip6_input(mp, off)
        rip6src.sin6_family = AF_INET6;
        rip6src.sin6_len = sizeof(struct sockaddr_in6);
        rip6src.sin6_addr = ip6->ip6_src;
-       if (sa6_recoverscope(&rip6src, TRUE)) 
+       if (sa6_recoverscope(&rip6src, TRUE))
                return (IPPROTO_DONE);
 
-       lck_rw_lock_shared(ripcbinfo.mtx);
+       lck_rw_lock_shared(ripcbinfo.ipi_lock);
        LIST_FOREACH(in6p, &ripcb, inp_list)
        {
                if ((in6p->inp_vflag & INP_IPV6) == 0)
@@ -2102,12 +2059,17 @@ icmp6_rip6_input(mp, off)
                    && ICMP6_FILTER_WILLBLOCK(icmp6->icmp6_type,
                                 in6p->in6p_icmp6filt))
                        continue;
+
+               if (inp_restricted_recv(in6p, ifp))
+                       continue;
+
                if (last) {
                        struct  mbuf *n;
                        if ((n = m_copy(m, 0, (int)M_COPYALL)) != NULL) {
-                               if ((last->in6p_flags & IN6P_CONTROLOPTS) != 0 ||
+                               if ((last->in6p_flags & INP_CONTROLOPTS) != 0 ||
                                    (last->in6p_socket->so_options & SO_TIMESTAMP) != 0 ||
-                                   (last->in6p_socket->so_options & SO_TIMESTAMP_MONOTONIC) != 0) {
+                                   (last->in6p_socket->so_options & SO_TIMESTAMP_MONOTONIC) != 0 ||
+                                       (last->in6p_socket->so_options & SO_TIMESTAMP_CONTINUOUS) != 0) {
                                        ret = ip6_savecontrol(last, n, &opts);
                                        if (ret != 0) {
                                                m_freem(n);
@@ -2132,7 +2094,8 @@ icmp6_rip6_input(mp, off)
        if (last) {
                if ((last->in6p_flags & INP_CONTROLOPTS) != 0 ||
                    (last->in6p_socket->so_options & SO_TIMESTAMP) != 0 ||
-                   (last->in6p_socket->so_options & SO_TIMESTAMP_MONOTONIC) != 0) {
+                   (last->in6p_socket->so_options & SO_TIMESTAMP_MONOTONIC) != 0 ||
+                       (last->in6p_socket->so_options & SO_TIMESTAMP_CONTINUOUS) != 0) {
                        ret = ip6_savecontrol(last, m, &opts);
                        if (ret != 0) {
                                goto error;
@@ -2148,11 +2111,11 @@ icmp6_rip6_input(mp, off)
        } else {
                goto error;
        }
-       lck_rw_done(ripcbinfo.mtx);
+       lck_rw_done(ripcbinfo.ipi_lock);
        return IPPROTO_DONE;
 
 error:
-       lck_rw_done(ripcbinfo.mtx);
+       lck_rw_done(ripcbinfo.ipi_lock);
        m_freem(m);
        m_freem(opts);
        ip6stat.ip6s_delivered--;
@@ -2164,10 +2127,9 @@ error:
  * OFF points to the icmp6 header, counted from the top of the mbuf.
  */
 void
-icmp6_reflect(m, off)
-       struct  mbuf *m;
-       size_t off;
+icmp6_reflect(struct mbuf *m, size_t off)
 {
+       struct mbuf *m_ip6hdr = m;
        struct ip6_hdr *ip6;
        struct icmp6_hdr *icmp6;
        struct in6_ifaddr *ia;
@@ -2176,12 +2138,18 @@ icmp6_reflect(m, off)
        int type, code;
        struct ifnet *outif = NULL;
        struct sockaddr_in6 sa6_src, sa6_dst;
-       struct nd_ifinfo *ndi;
+       struct nd_ifinfo *ndi = NULL;
        u_int32_t oflow;
-       struct ip6_out_args ip6oa =
-           { IFSCOPE_NONE, { 0 }, IP6OAF_SELECT_SRCIF | IP6OAF_BOUND_SRCADDR };
+       struct ip6_out_args ip6oa;
+
+       bzero(&ip6oa, sizeof(ip6oa));
+       ip6oa.ip6oa_boundif = IFSCOPE_NONE;
+       ip6oa.ip6oa_flags = IP6OAF_SELECT_SRCIF | IP6OAF_BOUND_SRCADDR |
+           IP6OAF_INTCOPROC_ALLOWED | IP6OAF_AWDL_UNRESTRICTED;
+       ip6oa.ip6oa_sotc = SO_TC_UNSPEC;
+       ip6oa.ip6oa_netsvctype = _NET_SERVICE_TYPE_UNSPEC;
 
-       if ((m->m_flags & M_PKTHDR) && m->m_pkthdr.rcvif != NULL) {
+       if (!(m->m_pkthdr.pkt_flags & PKTF_LOOP) && m->m_pkthdr.rcvif != NULL) {
                ip6oa.ip6oa_boundif = m->m_pkthdr.rcvif->if_index;
                ip6oa.ip6oa_flags |= IP6OAF_BOUND_IF;
        }
@@ -2191,7 +2159,7 @@ icmp6_reflect(m, off)
                nd6log((LOG_DEBUG,
                    "sanity fail: off=%lx, sizeof(ip6)=%lx in %s:%d\n",
                    (u_int32_t)off, (u_int32_t)sizeof(struct ip6_hdr),
-                   __FILE__, __LINE__));
+                   __func__, __LINE__));
                goto bad;
        }
 
@@ -2199,10 +2167,6 @@ icmp6_reflect(m, off)
         * If there are extra headers between IPv6 and ICMPv6, strip
         * off that header first.
         */
-#if DIAGNOSTIC
-       if (sizeof(struct ip6_hdr) + sizeof(struct icmp6_hdr) > MHLEN)
-               panic("assumption failed in icmp6_reflect");
-#endif
        if (off > sizeof(struct ip6_hdr)) {
                size_t l;
                struct ip6_hdr nip6;
@@ -2212,7 +2176,7 @@ icmp6_reflect(m, off)
                m_adj(m, l);
                l = sizeof(struct ip6_hdr) + sizeof(struct icmp6_hdr);
                if (m->m_len < l) {
-                       if ((m = m_pullup(m, l)) == NULL)
+                       if ((m_ip6hdr = m_pulldown(m, 0, l, NULL)) == NULL)
                                return;
                }
                bcopy((caddr_t)&nip6, mtod(m, caddr_t), sizeof(nip6));
@@ -2220,12 +2184,12 @@ icmp6_reflect(m, off)
                size_t l;
                l = sizeof(struct ip6_hdr) + sizeof(struct icmp6_hdr);
                if (m->m_len < l) {
-                       if ((m = m_pullup(m, l)) == NULL)
+                       if ((m_ip6hdr = m_pulldown(m, 0, l, NULL)) == NULL)
                                return;
                }
        }
        plen = m->m_pkthdr.len - sizeof(struct ip6_hdr);
-       ip6 = mtod(m, struct ip6_hdr *);
+       ip6 = mtod(m_ip6hdr, struct ip6_hdr *);
        ip6->ip6_nxt = IPPROTO_ICMPV6;
        icmp6 = (struct icmp6_hdr *)(ip6 + 1);
        type = icmp6->icmp6_type; /* keep type for statistics */
@@ -2270,7 +2234,7 @@ icmp6_reflect(m, off)
        for (ia = in6_ifaddrs; ia; ia = ia->ia_next) {
                IFA_LOCK(&ia->ia_ifa);
                if (IN6_ARE_ADDR_EQUAL(&t, &ia->ia_addr.sin6_addr) &&
-                   (ia->ia6_flags & (IN6_IFF_ANYCAST|IN6_IFF_NOTREADY)) == 0) {
+                   (ia->ia6_flags & (IN6_IFF_ANYCAST|IN6_IFF_NOTREADY|IN6_IFF_CLAT46)) == 0) {
                        IFA_UNLOCK(&ia->ia_ifa);
                        src = &t;
                        break;
@@ -2278,10 +2242,14 @@ icmp6_reflect(m, off)
                IFA_UNLOCK(&ia->ia_ifa);
        }
        lck_rw_done(&in6_ifaddr_rwlock);
-       if (ia == NULL && IN6_IS_ADDR_LINKLOCAL(&t) && (m->m_flags & M_LOOP)) {
+       if (ia == NULL && IN6_IS_ADDR_LINKLOCAL(&t) &&
+           ((m->m_flags & M_LOOP) || (m->m_pkthdr.pkt_flags & PKTF_LOOP))) {
                /*
                 * This is the case if the dst is our link-local address
-                * and the sender is also ourselves.
+                * and the sender is also ourselves.  Here we test for both
+                * M_LOOP and PKTF_LOOP, since the former may have been set
+                * in ip6_output() and that we get here as part of callling
+                * ip6_process_hopopts().  See comments in <sys/mbuf.h>
                 */
                src = &t;
        }
@@ -2309,8 +2277,7 @@ icmp6_reflect(m, off)
                 */
                src = in6_selectsrc(&sin6, NULL, NULL, &ro, &outif,
                    &src_storage, ip6oa.ip6oa_boundif, &e);
-               if (ro.ro_rt)
-                       rtfree(ro.ro_rt); /* XXX: we could use this */
+               ROUTE_RELEASE(&ro);
                if (src == NULL) {
                        nd6log((LOG_DEBUG,
                            "icmp6_reflect: source can't be determined: "
@@ -2328,7 +2295,6 @@ icmp6_reflect(m, off)
                ip6->ip6_flow |= (oflow & htonl(0x0ff00000));
        }
        ip6->ip6_nxt = IPPROTO_ICMPV6;
-       lck_rw_lock_shared(nd_if_rwlock);
        if (outif != NULL && (ndi = ND_IFINFO(outif)) != NULL &&
            ndi->initialized) {
                lck_mtx_lock(&ndi->lock);
@@ -2345,28 +2311,21 @@ icmp6_reflect(m, off)
        } else {
                ip6->ip6_hlim = ip6_defhlim;
        }
-       lck_rw_done(nd_if_rwlock);
        /* Use the same traffic class as in the request to match IPv4 */
        icmp6->icmp6_cksum = 0;
        icmp6->icmp6_cksum = in6_cksum(m, IPPROTO_ICMPV6,
-                                       sizeof(struct ip6_hdr), plen);
+           sizeof(struct ip6_hdr), plen);
 
        /*
         * XXX option handling
         */
-
        m->m_flags &= ~(M_BCAST|M_MCAST);
-#if IPSEC
-       /* Don't lookup socket */
-       if (ipsec_bypass == 0)
-               (void)ipsec_setsocket(m, NULL);
-#endif /*IPSEC*/
 
        if (outif != NULL) {
                ifnet_release(outif);
                outif = NULL;
        }
-       m->m_pkthdr.rcvif = NULL;
+
        m->m_pkthdr.csum_data = 0;
        m->m_pkthdr.csum_flags = 0;
        ip6_output(m, NULL, NULL, IPV6_OUTARGS, NULL, &outif, &ip6oa);
@@ -2384,10 +2343,9 @@ bad:
 }
 
 static const char *
-icmp6_redirect_diag(src6, dst6, tgt6)
-       struct in6_addr *src6;
-       struct in6_addr *dst6;
-       struct in6_addr *tgt6;
+icmp6_redirect_diag(struct in6_addr *src6,
+                   struct in6_addr *dst6,
+                   struct in6_addr *tgt6)
 {
        static char buf[1024];
        snprintf(buf, sizeof(buf), "(src=%s dst=%s tgt=%s)",
@@ -2396,9 +2354,7 @@ icmp6_redirect_diag(src6, dst6, tgt6)
 }
 
 void
-icmp6_redirect_input(m, off)
-       struct mbuf *m;
-       int off;
+icmp6_redirect_input(struct mbuf *m, int off)
 {
        struct ifnet *ifp = m->m_pkthdr.rcvif;
        struct ip6_hdr *ip6 = mtod(m, struct ip6_hdr *);
@@ -2590,15 +2546,7 @@ icmp6_redirect_input(m, off)
        sdst.sin6_len = sizeof(struct sockaddr_in6);
        bcopy(&reddst6, &sdst.sin6_addr, sizeof(struct in6_addr));
 
-       /*
-         * Radar 6843900
-        * Release the IPv6 domain lock because we are going to take domain_proto_mtx
-        * and could otherwise cause a deadlock with other threads taking these locks 
-        * in the reverse order -- e.g. frag6_slowtimo() from pfslowtimo()
-        */
-       lck_mtx_unlock(inet6_domain_mutex);
        pfctlinput(PRC_REDIRECT_HOST, (struct sockaddr *)&sdst);
-       lck_mtx_lock(inet6_domain_mutex);
 #if IPSEC
        key_sa_routechange((struct sockaddr *)&sdst);
 #endif
@@ -2614,9 +2562,7 @@ icmp6_redirect_input(m, off)
 }
 
 void
-icmp6_redirect_output(m0, rt)
-       struct mbuf *m0;
-       struct rtentry *rt;
+icmp6_redirect_output(struct mbuf *m0, struct rtentry *rt)
 {
        struct ifnet *ifp;      /* my outgoing interface */
        struct in6_addr ifp_ll6;
@@ -2629,8 +2575,13 @@ icmp6_redirect_output(m0, rt)
        u_char *p;
        struct ifnet *outif = NULL;
        struct sockaddr_in6 src_sa;
-       struct ip6_out_args ip6oa =
-           { IFSCOPE_NONE, { 0 }, IP6OAF_SELECT_SRCIF | IP6OAF_BOUND_SRCADDR };
+       struct ip6_out_args ip6oa;
+
+       bzero(&ip6oa, sizeof(ip6oa));
+       ip6oa.ip6oa_boundif = IFSCOPE_NONE;
+       ip6oa.ip6oa_flags = IP6OAF_SELECT_SRCIF | IP6OAF_BOUND_SRCADDR;
+       ip6oa.ip6oa_sotc = SO_TC_UNSPEC;
+       ip6oa.ip6oa_netsvctype = _NET_SERVICE_TYPE_UNSPEC;
 
        icmp6_errcount(&icmp6stat.icp6s_outerrhist, ND_REDIRECT, 0);
 
@@ -2702,8 +2653,8 @@ icmp6_redirect_output(m0, rt)
                /* get ip6 linklocal address for ifp(my outgoing interface). */
                struct in6_ifaddr *ia;
                if ((ia = in6ifa_ifpforlinklocal(ifp,
-                                                IN6_IFF_NOTREADY|
-                                                IN6_IFF_ANYCAST)) == NULL)
+                   IN6_IFF_NOTREADY|
+                   IN6_IFF_ANYCAST)) == NULL)
                        goto fail;
                IFA_LOCK(&ia->ia_ifa);
                ifp_ll6 = ia->ia_addr.sin6_addr;
@@ -2846,7 +2797,7 @@ nolladdropt:;
         * and truncates if not.
         */
        if (m0->m_next || m0->m_pkthdr.len != m0->m_len)
-               panic("assumption failed in %s:%d\n", __FILE__, __LINE__);
+               panic("assumption failed in %s:%d\n", __func__, __LINE__);
 
        if (len - sizeof(*nd_opt_rh) < m0->m_pkthdr.len) {
                /* not enough room, truncate */
@@ -2898,12 +2849,6 @@ noredhdropt:;
                = in6_cksum(m, IPPROTO_ICMPV6, sizeof(*ip6), ntohs(ip6->ip6_plen));
 
        /* send the packet to outside... */
-#if IPSEC
-       /* Don't lookup socket */
-       if (ipsec_bypass == 0)
-               (void)ipsec_setsocket(m, NULL);
-#endif /*IPSEC*/
-
        ip6oa.ip6oa_boundif = ifp->if_index;
        ip6oa.ip6oa_flags |= IP6OAF_BOUND_IF;
 
@@ -2930,9 +2875,7 @@ fail:
  * ICMPv6 socket option processing.
  */
 int
-icmp6_ctloutput(so, sopt)
-       struct socket *so;
-       struct sockopt *sopt;
+icmp6_ctloutput(struct socket *so, struct sockopt *sopt)
 {
        int error = 0;
        int optlen;
@@ -2968,7 +2911,7 @@ icmp6_ctloutput(so, sopt)
                        }
 
                        if (optlen == 0) {
-                               /* According to RFC 3542, an installed filter can be 
+                               /* According to RFC 3542, an installed filter can be
                                 * cleared by issuing a setsockopt for ICMP6_FILTER
                                 * with a zero length.
                                 */
@@ -3079,15 +3022,23 @@ icmp6_dgram_send(struct socket *so, int flags, struct mbuf *m,
        struct sockaddr_in6 *dst = (struct sockaddr_in6 *)(void *)nam;
        struct icmp6_hdr *icmp6;
 
+       if (inp == NULL
+#if NECP
+               || (necp_socket_should_use_flow_divert(inp))
+#endif /* NECP */
+               ) {
+               error = (inp == NULL ? EINVAL : EPROTOTYPE);
+               goto bad;
+       }
+
        if (kauth_cred_issuser(so->so_cred))
-               return rip6_output(m, so, (struct sockaddr_in6 *)(void *)nam,
-                   control, 0);
+               return (rip6_output(m, so, SIN6(nam), control, 0));
 
        /* always copy sockaddr to avoid overwrites */
        if (so->so_state & SS_ISCONNECTED) {
-               if (nam) {
-                       m_freem(m);
-                       return EISCONN;
+               if (nam != NULL) {
+                       error = EISCONN;
+                       goto bad;
                }
                /* XXX */
                bzero(&tmp, sizeof(tmp));
@@ -3098,8 +3049,8 @@ icmp6_dgram_send(struct socket *so, int flags, struct mbuf *m,
                dst = &tmp;
        } else {
                if (nam == NULL) {
-                       m_freem(m);
-                       return ENOTCONN;
+                       error = ENOTCONN;
+                       goto bad;
                }
                tmp = *(struct sockaddr_in6 *)(void *)nam;
                dst = &tmp;
@@ -3108,7 +3059,7 @@ icmp6_dgram_send(struct socket *so, int flags, struct mbuf *m,
        /*
         * For an ICMPv6 packet, we should know its type and code
         */
-       if (so->so_proto->pr_protocol == IPPROTO_ICMPV6) {
+       if (SOCK_PROTO(so) == IPPROTO_ICMPV6) {
                if (m->m_len < sizeof(struct icmp6_hdr) &&
                        (m = m_pullup(m, sizeof(struct icmp6_hdr))) == NULL) {
                                error = ENOBUFS;
@@ -3120,10 +3071,11 @@ icmp6_dgram_send(struct socket *so, int flags, struct mbuf *m,
                 * Allow only to send echo request and node information request
                 * See RFC 2463 for Echo Request Message format
                 */
-               if ((icmp6->icmp6_type == ICMP6_ECHO_REQUEST && icmp6->icmp6_code == 0) ||
-                       (icmp6->icmp6_type == ICMP6_NI_QUERY &&
-                        (icmp6->icmp6_code == ICMP6_NI_SUBJ_IPV6 ||
-                         icmp6->icmp6_code == ICMP6_NI_SUBJ_FQDN))) {
+               if ((icmp6->icmp6_type == ICMP6_ECHO_REQUEST &&
+                   icmp6->icmp6_code == 0) ||
+                   (icmp6->icmp6_type == ICMP6_NI_QUERY &&
+                   (icmp6->icmp6_code == ICMP6_NI_SUBJ_IPV6 ||
+                   icmp6->icmp6_code == ICMP6_NI_SUBJ_FQDN))) {
                        /* Good */
                        ;
                } else {
@@ -3134,15 +3086,20 @@ icmp6_dgram_send(struct socket *so, int flags, struct mbuf *m,
 
 #if ENABLE_DEFAULT_SCOPE
        if (dst->sin6_scope_id == 0) {  /* not change if specified  */
-                       dst->sin6_scope_id = scope6_addr2default(&dst->sin6_addr);
+               dst->sin6_scope_id = scope6_addr2default(&dst->sin6_addr);
        }
 #endif
 
-       return rip6_output(m, so, (struct sockaddr_in6 *)(void *)nam,
-          control, 0);
+       return (rip6_output(m, so, dst, control, 0));
 bad:
-       m_freem(m);
-       return error;
+       VERIFY(error != 0);
+
+       if (m != NULL)
+               m_freem(m);
+       if (control != NULL)
+               m_freem(control);
+
+       return (error);
 }
 
 /* Like rip6_attach but without root privilege enforcement */
@@ -3179,95 +3136,30 @@ icmp6_dgram_attach(struct socket *so, int proto, struct proc *p)
 }
 
 
-
-#ifndef HAVE_PPSRATECHECK
-#ifndef timersub
-#define        timersub(tvp, uvp, vvp)                                         \
-       do {                                                            \
-               (vvp)->tv_sec = (tvp)->tv_sec - (uvp)->tv_sec;          \
-               (vvp)->tv_usec = (tvp)->tv_usec - (uvp)->tv_usec;       \
-               if ((vvp)->tv_usec < 0) {                               \
-                       (vvp)->tv_sec--;                                \
-                       (vvp)->tv_usec += 1000000;                      \
-               }                                                       \
-       } while (0)
-#endif
-
-/*
- * ppsratecheck(): packets (or events) per second limitation.
- */
-static int
-ppsratecheck(lasttime, curpps, maxpps)
-       struct timeval *lasttime;
-       int *curpps;
-       int maxpps;     /* maximum pps allowed */
-{
-       struct timeval tv, delta;
-       int rv;
-
-       microtime(&tv);
-
-       timersub(&tv, lasttime, &delta);
-
-       /*
-        * Check for 0,0 so that the message will be seen at least once.
-        * If more than one second has passed since the last update of
-        * lasttime, reset the counter.
-        *
-        * we do increment *curpps even in *curpps < maxpps case, as some may
-        * try to use *curpps for stat purposes as well.
-        */
-       if ((lasttime->tv_sec == 0 && lasttime->tv_usec == 0) ||
-           delta.tv_sec >= 1) {
-               *lasttime = tv;
-               *curpps = 0;
-               rv = 1;
-       } else if (maxpps < 0)
-               rv = 1;
-       else if (*curpps < maxpps)
-               rv = 1;
-       else
-               rv = 0;
-
-#if 1 /* DIAGNOSTIC? */
-       /* be careful about wrap-around */
-       if (*curpps + 1 > 0)
-               *curpps = *curpps + 1;
-#else
-       /*
-        * assume that there's not too many calls to this function.
-        * not sure if the assumption holds, as it depends on *caller's*
-        * behavior, not the behavior of this function.
-        * IMHO it is wrong to make assumption on the caller's behavior,
-        * so the above #if is #if 1, not #ifdef DIAGNOSTIC.
-        */
-       *curpps = *curpps + 1;
-#endif
-
-       return (rv);
-}
-#endif
-
 /*
  * Perform rate limit check.
  * Returns 0 if it is okay to send the icmp6 packet.
  * Returns 1 if the router SHOULD NOT send this icmp6 packet due to rate
  * limitation.
  *
- * XXX per-destination/type check necessary?
+ * XXX per-destination check necessary?
  */
 static int
 icmp6_ratelimit(
        __unused const struct in6_addr *dst,    /* not used at this moment */
-       __unused const int type,                        /* not used at this moment */
-       __unused const int code)                        /* not used at this moment */
+       const int type,
+       __unused const int code)
 {
        int ret;
 
        ret = 0;        /* okay to send */
 
        /* PPS limit */
-       if (!ppsratecheck(&icmp6errppslim_last, &icmp6errpps_count,
+       if (type == ND_ROUTER_ADVERT) {
+               if (!ppsratecheck(&icmp6rappslim_last, &icmp6rapps_count,
+                   icmp6rappslim))
+                       ret++;
+       } else if (!ppsratecheck(&icmp6errppslim_last, &icmp6errpps_count,
            icmp6errppslim)) {
                /* The packet is subject to rate limit */
                ret++;