Skip site navigation (1)Skip section navigation (2) 
Date:      Sat, 6 Nov 2010 15:56:44 +0000 (UTC)
From:      "Bjoern A. Zeeb" <bz@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-7@freebsd.org
Subject:   svn commit: r214891 - in stable/7/sys: netinet netinet6 netipsec
Message-ID:  <201011061556.oA6FuiHb036245@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help 
Author: bz
Date: Sat Nov  6 15:56:44 2010
New Revision: 214891
URL: http://svn.freebsd.org/changeset/base/214891

Log:
  MFC r214250:
  
    Make the IPsec SADB embedded route cache a union to be able to hold both the
    legacy and IPv6 route destination address.
    Previously in case of IPv6, there was a memory overwrite due to not enough
    space for the IPv6 address.
  
  PR:		kern/122565

Modified:
  stable/7/sys/netinet/ip_ipsec.c
  stable/7/sys/netinet6/ip6_ipsec.c
  stable/7/sys/netipsec/ipsec_output.c
  stable/7/sys/netipsec/key.c
  stable/7/sys/netipsec/keydb.h
Directory Properties:
  stable/7/sys/   (props changed)
  stable/7/sys/cddl/contrib/opensolaris/   (props changed)
  stable/7/sys/contrib/dev/acpica/   (props changed)
  stable/7/sys/contrib/pf/   (props changed)

Modified: stable/7/sys/netinet/ip_ipsec.c
==============================================================================
--- stable/7/sys/netinet/ip_ipsec.c	Sat Nov  6 15:49:59 2010	(r214890)
+++ stable/7/sys/netinet/ip_ipsec.c	Sat Nov  6 15:56:44 2010	(r214891)
@@ -220,7 +220,7 @@ ip_ipsec_mtu(struct mbuf *m, int mtu)
 		if (sp->req != NULL &&
 		    sp->req->sav != NULL &&
 		    sp->req->sav->sah != NULL) {
-			ro = &sp->req->sav->sah->sa_route;
+			ro = &sp->req->sav->sah->route_cache.sa_route;
 			if (ro->ro_rt && ro->ro_rt->rt_ifp) {
 				mtu =
 				    ro->ro_rt->rt_rmx.rmx_mtu ?

Modified: stable/7/sys/netinet6/ip6_ipsec.c
==============================================================================
--- stable/7/sys/netinet6/ip6_ipsec.c	Sat Nov  6 15:49:59 2010	(r214890)
+++ stable/7/sys/netinet6/ip6_ipsec.c	Sat Nov  6 15:56:44 2010	(r214891)
@@ -346,7 +346,7 @@ ip6_ipsec_mtu(struct mbuf *m)
 		if (sp->req != NULL &&
 		    sp->req->sav != NULL &&
 		    sp->req->sav->sah != NULL) {
-			ro = &sp->req->sav->sah->sa_route;
+			ro = &sp->req->sav->sah->route_cache.sa_route;
 			if (ro->ro_rt && ro->ro_rt->rt_ifp) {
 				mtu =
 				    ro->ro_rt->rt_rmx.rmx_mtu ?

Modified: stable/7/sys/netipsec/ipsec_output.c
==============================================================================
--- stable/7/sys/netipsec/ipsec_output.c	Sat Nov  6 15:49:59 2010	(r214890)
+++ stable/7/sys/netipsec/ipsec_output.c	Sat Nov  6 15:56:44 2010	(r214891)
@@ -773,7 +773,8 @@ ipsec6_output_tunnel(struct ipsec_output
 		}
 		ip6 = mtod(m, struct ip6_hdr *);
 
-		state->ro = &isr->sav->sah->sa_route;
+		state->ro =
+		    (struct route *)&isr->sav->sah->route_cache.sin6_route;
 		state->dst = (struct sockaddr *)&state->ro->ro_dst;
 		dst6 = (struct sockaddr_in6 *)state->dst;
 		if (state->ro->ro_rt

Modified: stable/7/sys/netipsec/key.c
==============================================================================
--- stable/7/sys/netipsec/key.c	Sat Nov  6 15:49:59 2010	(r214890)
+++ stable/7/sys/netipsec/key.c	Sat Nov  6 15:56:44 2010	(r214891)
@@ -2674,9 +2674,9 @@ key_delsah(sah)
 		/* remove from tree of SA index */
 		if (__LIST_CHAINED(sah))
 			LIST_REMOVE(sah, chain);
-		if (sah->sa_route.ro_rt) {
-			RTFREE(sah->sa_route.ro_rt);
-			sah->sa_route.ro_rt = (struct rtentry *)NULL;
+		if (sah->route_cache.sa_route.ro_rt) {
+			RTFREE(sah->route_cache.sa_route.ro_rt);
+			sah->route_cache.sa_route.ro_rt = (struct rtentry *)NULL;
 		}
 		free(sah, M_IPSEC_SAH);
 	}
@@ -7196,7 +7196,7 @@ key_sa_routechange(dst)
 
 	SAHTREE_LOCK();
 	LIST_FOREACH(sah, &sahtree, chain) {
-		ro = &sah->sa_route;
+		ro = &sah->route_cache.sa_route;
 		if (ro->ro_rt && dst->sa_len == ro->ro_dst.sa_len
 		 && bcmp(dst, &ro->ro_dst, dst->sa_len) == 0) {
 			RTFREE(ro->ro_rt);

Modified: stable/7/sys/netipsec/keydb.h
==============================================================================
--- stable/7/sys/netipsec/keydb.h	Sat Nov  6 15:49:59 2010	(r214890)
+++ stable/7/sys/netipsec/keydb.h	Sat Nov  6 15:56:44 2010	(r214891)
@@ -85,6 +85,12 @@ struct seclifetime {
 	u_int64_t usetime;
 };
 
+union sa_route_union {
+	struct route		sa_route;
+	struct route		sin_route;	/* Duplicate for consistency. */
+	struct route_in6	sin6_route;
+};
+
 /* Security Association Data Base */
 struct secashead {
 	LIST_ENTRY(secashead) chain;
@@ -100,7 +106,7 @@ struct secashead {
 					/* SA chain */
 					/* The first of this list is newer SA */
 
-	struct route sa_route;		/* route cache */
+	union sa_route_union route_cache;
 };
 
 struct xformsw;

Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201011061556.oA6FuiHb036245>