From owner-dev-commits-src-all@freebsd.org Wed Jan 27 20:24:22 2021 Return-Path: Delivered-To: dev-commits-src-all@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id 0A2514E8744; Wed, 27 Jan 2021 20:24:22 +0000 (UTC) (envelope-from Michael.Tuexen@macmic.franken.de) Received: from drew.franken.de (mail-n.franken.de [193.175.24.27]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "*.franken.de", Issuer "Sectigo RSA Domain Validation Secure Server CA" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4DQw792j7xz3KTw; Wed, 27 Jan 2021 20:24:19 +0000 (UTC) (envelope-from Michael.Tuexen@macmic.franken.de) Received: from [IPv6:2a02:8109:1140:c3d:1047:2f49:ddfe:569c] (unknown [IPv6:2a02:8109:1140:c3d:1047:2f49:ddfe:569c]) (Authenticated sender: macmic) by mail-n.franken.de (Postfix) with ESMTPSA id B48A4721BE010; Wed, 27 Jan 2021 21:24:15 +0100 (CET) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.40.0.2.32\)) Subject: Re: git: 24a8f6d36996 - main - When we are about to send down to the driver layer we need to make sure that the m_nextpkt field is NULL else the lower layers may do unwanted things. From: Michael Tuexen In-Reply-To: <202101271854.10RIsN1E083640@gitrepo.freebsd.org> Date: Wed, 27 Jan 2021 21:24:12 +0100 Cc: "src-committers@freebsd.org" , "dev-commits-src-all@freebsd.org" , "dev-commits-src-main@freebsd.org" Content-Transfer-Encoding: quoted-printable Message-Id: <9F54A720-4A55-48E6-88BC-CFEFC48F79B0@macmic.franken.de> References: <202101271854.10RIsN1E083640@gitrepo.freebsd.org> To: Randall Stewart X-Mailer: Apple Mail (2.3654.40.0.2.32) X-Spam-Status: No, score=-2.9 required=5.0 tests=ALL_TRUSTED,BAYES_00 autolearn=disabled version=3.4.1 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on mail-n.franken.de X-Rspamd-Queue-Id: 4DQw792j7xz3KTw X-Spamd-Bar: ---- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-4.00 / 15.00]; REPLY(-4.00)[] X-BeenThere: dev-commits-src-all@freebsd.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Commit messages for all branches of the src repository List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 27 Jan 2021 20:24:22 -0000 > On 27. Jan 2021, at 19:54, Randall Stewart wrote: >=20 > The branch main has been updated by rrs: >=20 > URL: = https://cgit.FreeBSD.org/src/commit/?id=3D24a8f6d369962f189ad808f538029179= b1e7dc2f >=20 > commit 24a8f6d369962f189ad808f538029179b1e7dc2f > Author: Randall Stewart > AuthorDate: 2021-01-27 18:32:52 +0000 > Commit: Randall Stewart > CommitDate: 2021-01-27 18:52:44 +0000 >=20 > When we are about to send down to the driver layer > we need to make sure that the m_nextpkt field is NULL > else the lower layers may do unwanted things. >=20 > Reviewed By: gallatin, melifaro > Differential Revision: https://reviews.freebsd.org/D28377 Any plans to MFC this to stable/13 and or stable/12? Best regards Michael > --- > sys/netinet6/nd6.c | 1 + > 1 file changed, 1 insertion(+) >=20 > diff --git a/sys/netinet6/nd6.c b/sys/netinet6/nd6.c > index 386eb7cca922..497c0bfc10e8 100644 > --- a/sys/netinet6/nd6.c > +++ b/sys/netinet6/nd6.c > @@ -2460,6 +2460,7 @@ nd6_flush_holdchain(struct ifnet *ifp, struct = mbuf *chain, > while (m_head) { > m =3D m_head; > m_head =3D m_head->m_nextpkt; > + m->m_nextpkt =3D NULL; > error =3D nd6_output_ifp(ifp, ifp, m, dst, NULL); > } >=20