Date: Tue, 12 Dec 2017 17:11:01 +0000 From: Glen Barber <gjb@FreeBSD.org> To: Michael Tuexen <tuexen@FreeBSD.org> Cc: Konstantin Belousov <kib@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-11@freebsd.org Subject: Re: svn commit: r324686 - stable/11/sys/netinet Message-ID: <20171212171101.GM17600@FreeBSD.org> In-Reply-To: <20171203133946.GJ2272@kib.kiev.ua> References: <201710171242.v9HCgHG4030707@repo.freebsd.org> <20171203133946.GJ2272@kib.kiev.ua>
next in thread | previous in thread | raw e-mail | index | archive | help
--W4pDZ/VvazBYHhxQ Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Dec 03, 2017 at 03:39:46PM +0200, Konstantin Belousov wrote: > On Tue, Oct 17, 2017 at 12:42:17PM +0000, Michael Tuexen wrote: > > Author: tuexen > > Date: Tue Oct 17 12:42:17 2017 > > New Revision: 324686 > > URL: https://svnweb.freebsd.org/changeset/base/324686 > >=20 > > Log: > > MFC r322648: > > =20 > > Ensure inp_vflag is consistently set for TCP endpoints. > > =20 > > Make sure that the flags INP_IPV4 and INP_IPV6 are consistently set > > for inpcbs used for TCP sockets, no matter if the setting is derived > > from the net.inet6.ip6.v6only sysctl or the IPV6_V6ONLY socket option. > > For UDP this was already done right. > > =20 > > PR: 221385 > >=20 > > Modified: > > stable/11/sys/netinet/tcp_usrreq.c > > Directory Properties: > > stable/11/ (props changed) > >=20 > > Modified: stable/11/sys/netinet/tcp_usrreq.c > > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D > > --- stable/11/sys/netinet/tcp_usrreq.c Tue Oct 17 11:20:32 2017 (r32468= 5) > > +++ stable/11/sys/netinet/tcp_usrreq.c Tue Oct 17 12:42:17 2017 (r32468= 6) > > @@ -1899,6 +1899,8 @@ tcp_attach(struct socket *so) > > #ifdef INET6 > > if (inp->inp_vflag & INP_IPV6PROTO) { > > inp->inp_vflag |=3D INP_IPV6; > > + if ((inp->inp_flags & IN6P_IPV6_V6ONLY) =3D=3D 0) > > + inp->inp_vflag |=3D INP_IPV4; > > inp->in6p_hops =3D -1; /* use kernel default */ > > } > > else >=20 > I highly suspect that this commit causes the breakage. Example is taken > from the 'nestat -f inet6 -n' output, but in fact the problem is much > more serious, e.g. squid acls are broken as well: > tcp6 0 0 2001:470:1f14:13.23047 2a03:2880:f003:c.443 ESTABLI= SHED > tcp46 0 0 0.0.0.1.22 69.223.183.85.58766 ESTABLI= SHED >=20 > The tcp46 and 0.0.0.1 values in the second line are clearly nonsensical. >=20 Hi Michael, Following up on this, as there has been no reply from you. Glen --W4pDZ/VvazBYHhxQ Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEjRJAPC5sqwhs9k2jAxRYpUeP4pMFAlowDaUACgkQAxRYpUeP 4pMt8w//YS0kijGEY7ngu6Bvy9A3F0LRepy7vzl2f5HW+so30qX/g6D1K3OqBMnH i/YbaIF5ts8/+PqMTt3ImwP103LIlVl9RbJtJftHE9hnyfmMtWWz1IOvjRtqU4Iz OkkgbtMRYoCyG1HMSH8o0UJ2Lf1usyWYVgAAHfIXhnKN9h3uZ1jfKUjPdhehzKY7 EAXERJ4ab4DWHGS0/f+HXkRLQurSmY4gdW+d9Irl4LsmjHy5iU+lyIcW0l909CAe s9y1ElIxmrzTE8pkz5bestdwPHokxGEfDXM3ZzkfZS/QdM9g4U+Hzf98QKl12bfS Rsx3AAmZ1Pfy68cE1hHBOn3kM2riUlUXpenlWjVWwBl7lK8ono4HEl+idi8TKZlq 10loWmlk16vH9mn51CK2gPnZKteWLk5DolaSqBNl2QXFr4tn31iysoBYgHXG/cko HFksdOWsdRJUwVorvMCPBZaOeisCmZ6bGVi9cmk782/bnvz3ReL6MfKGroXpV6Vc rKmlCi7Fu6cpk9ry5khGJ4eEtkhs49kdWaF5sP9mLzMh+nFMbGzQS6EUYt3PiSYC GuhSrspVIKHLt2he2I3NRIPY94FXWecN4x7IVHmpy9xK/+Lij02h3J1ZggD4shij HI7zdKHvH7HnqAFXPt2vzpcQjzfFXorLIZS0493u2zbaoUXp9bQ= =Ongh -----END PGP SIGNATURE----- --W4pDZ/VvazBYHhxQ--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20171212171101.GM17600>