From owner-svn-src-all@freebsd.org Thu Jan 28 23:13:13 2016 Return-Path: Delivered-To: svn-src-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id A086A9D9538; Thu, 28 Jan 2016 23:13:13 +0000 (UTC) (envelope-from tuexen@freebsd.org) Received: from mail-n.franken.de (drew.ipv6.franken.de [IPv6:2001:638:a02:a001:20e:cff:fe4a:feaa]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client CN "*.franken.de", Issuer "COMODO RSA Domain Validation Secure Server CA" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 54395DE9; Thu, 28 Jan 2016 23:13:13 +0000 (UTC) (envelope-from tuexen@freebsd.org) Received: from [192.168.1.200] (p4FE315DD.dip0.t-ipconnect.de [79.227.21.221]) (Authenticated sender: macmic) by mail-n.franken.de (Postfix) with ESMTP id 5B8971C0B462F; Fri, 29 Jan 2016 00:13:09 +0100 (CET) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 9.2 \(3112\)) Subject: Re: svn commit: r294995 - head/sys/netinet From: Michael Tuexen In-Reply-To: Date: Fri, 29 Jan 2016 00:13:08 +0100 Cc: "src-committers@freebsd.org" , "svn-src-all@freebsd.org" , "svn-src-head@freebsd.org" Content-Transfer-Encoding: quoted-printable Message-Id: <7FF45A03-6FC8-4FFF-94E5-BC6212F5EF02@freebsd.org> References: <201601281605.u0SG5kYU027051@repo.freebsd.org> To: Oliver Pinter X-Mailer: Apple Mail (2.3112) X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.20 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 28 Jan 2016 23:13:13 -0000 > On 28 Jan 2016, at 23:24, Oliver Pinter = wrote: >=20 > On Thu, Jan 28, 2016 at 5:05 PM, Michael Tuexen = wrote: >> Author: tuexen >> Date: Thu Jan 28 16:05:46 2016 >> New Revision: 294995 >> URL: https://svnweb.freebsd.org/changeset/base/294995 >>=20 >> Log: >> Always look in the TCP pool. >> This fixes issues with a restarting peer when the listening >> 1-to-1 style socket is closed. >>=20 >> MFC after: 3 days >>=20 >> Modified: >> head/sys/netinet/sctp_input.c >> head/sys/netinet/sctp_pcb.c >>=20 >> Modified: head/sys/netinet/sctp_input.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 >> --- head/sys/netinet/sctp_input.c Thu Jan 28 15:44:14 2016 = (r294994) >> +++ head/sys/netinet/sctp_input.c Thu Jan 28 16:05:46 2016 = (r294995) >> @@ -5688,6 +5688,7 @@ sctp_common_input_processing(struct mbuf >> if (net->port =3D=3D 0) { >> sctp_pathmtu_adjustment(stcb, = net->mtu - sizeof(struct udphdr)); >> } >> +printf("Changing remote encaps port from %u to %u.\n", = ntohs(net->port), ntohs(port)); >=20 > Are these debug printf intended? No. Fixed in https://svnweb.freebsd.org/changeset/base/295021 Thanks for reporting! Best regards Michael >=20 >> net->port =3D port; >> } >> #endif >> @@ -5719,6 +5720,7 @@ sctp_common_input_processing(struct mbuf >> if (net->port =3D=3D 0) { >> sctp_pathmtu_adjustment(stcb, net->mtu - = sizeof(struct udphdr)); >> } >> +printf("Changing remote encaps port from %u to %u.\n", = ntohs(net->port), ntohs(port)); >> net->port =3D port; >> } >> #endif >> @@ -5831,6 +5833,7 @@ sctp_common_input_processing(struct mbuf >> if (net->port =3D=3D 0) { >> sctp_pathmtu_adjustment(stcb, = net->mtu - sizeof(struct udphdr)); >> } >> +printf("Changing remote encaps port from %u to %u.\n", = ntohs(net->port), ntohs(port)); >> net->port =3D port; >> } >> #endif >>=20 >> Modified: head/sys/netinet/sctp_pcb.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 >> --- head/sys/netinet/sctp_pcb.c Thu Jan 28 15:44:14 2016 = (r294994) >> +++ head/sys/netinet/sctp_pcb.c Thu Jan 28 16:05:46 2016 = (r294995) >> @@ -2256,7 +2256,6 @@ sctp_findassociation_addr(struct mbuf *m >> struct sctphdr *sh, struct sctp_chunkhdr *ch, >> struct sctp_inpcb **inp_p, struct sctp_nets **netp, uint32_t = vrf_id) >> { >> - int find_tcp_pool; >> struct sctp_tcb *stcb; >> struct sctp_inpcb *inp; >>=20 >> @@ -2268,25 +2267,13 @@ sctp_findassociation_addr(struct mbuf *m >> return (stcb); >> } >> } >> - find_tcp_pool =3D 0; >> - /* >> - * Don't consider INIT chunks since that breaks 1-to-1 = sockets: When >> - * a server closes the listener, incoming INIT chunks are not >> - * responsed by an INIT-ACK chunk. >> - */ >> - if ((ch->chunk_type !=3D SCTP_INITIATION_ACK) && >> - (ch->chunk_type !=3D SCTP_COOKIE_ACK) && >> - (ch->chunk_type !=3D SCTP_COOKIE_ECHO)) { >> - /* Other chunk types go to the tcp pool. */ >> - find_tcp_pool =3D 1; >> - } >> if (inp_p) { >> stcb =3D sctp_findassociation_addr_sa(src, dst, inp_p, = netp, >> - find_tcp_pool, vrf_id); >> + 1, vrf_id); >> inp =3D *inp_p; >> } else { >> stcb =3D sctp_findassociation_addr_sa(src, dst, &inp, = netp, >> - find_tcp_pool, vrf_id); >> + 1, vrf_id); >> } >> SCTPDBG(SCTP_DEBUG_PCB1, "stcb:%p inp:%p\n", (void *)stcb, = (void *)inp); >> if (stcb =3D=3D NULL && inp) { >> _______________________________________________ >> svn-src-head@freebsd.org mailing list >> https://lists.freebsd.org/mailman/listinfo/svn-src-head >> To unsubscribe, send any mail to = "svn-src-head-unsubscribe@freebsd.org" >=20