Date: Wed, 31 Oct 2001 08:34:12 +0800 From: "XuYifeng" <xuyifeng@peoplemail.com.cn> To: "Bill Paul" <wpaul@FreeBSD.org>, <cvs-committers@FreeBSD.org>, <cvs-all@FreeBSD.org> Subject: Re: cvs commit: src/sys/netinet ip_output.c Message-ID: <002b01c161a3$c3ed5900$ef01a8c0@davidwnt> References: <200110301815.f9UIFmh57547@freefall.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Thanks, can you MFC it? -- David Xu ----- Original Message -----=20 From: "Bill Paul" <wpaul@FreeBSD.org> To: <cvs-committers@FreeBSD.org>; <cvs-all@FreeBSD.org> Sent: Wednesday, October 31, 2001 2:15 AM Subject: cvs commit: src/sys/netinet ip_output.c > wpaul 2001/10/30 10:15:48 PST >=20 > Modified files: > sys/netinet ip_output.c=20 > Log: > Fix a (long standing?) bug in ip_output(): if ip_insertoptions() is > called and ip_output() encounters an error and bails (i.e. host > unreachable), we will leak an mbuf. This is because the code calls > m_freem(m0) after jumping to the bad: label at the end of the = function, > when it should be calling m_freem(m). (m0 is the original mbuf list > _without_ the options mbuf prepended.) > =20 > Obtained from: NetBSD > =20 > Revision Changes Path > 1.141 +1 -1 src/sys/netinet/ip_output.c >=20 > To Unsubscribe: send mail to majordomo@FreeBSD.org > with "unsubscribe cvs-all" in the body of the message >=20 To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?002b01c161a3$c3ed5900$ef01a8c0>