From owner-cvs-all Sat Dec 1 5:51:35 2001 Delivered-To: cvs-all@freebsd.org Received: from whale.sunbay.crimea.ua (whale.sunbay.crimea.ua [212.110.138.65]) by hub.freebsd.org (Postfix) with ESMTP id 7F5F337B416 for ; Sat, 1 Dec 2001 05:51:30 -0800 (PST) Received: (from ru@localhost) by whale.sunbay.crimea.ua (8.11.6/8.11.2) id fB1Dp9q46131; Sat, 1 Dec 2001 15:51:09 +0200 (EET) (envelope-from ru) Date: Sat, 1 Dec 2001 15:51:09 +0200 From: Ruslan Ermilov To: Maxime Henrion Cc: cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/netinet icmp_var.h ip_icmp.c ip_input.c ip_var.h src/usr.bin/netstat inet.c Message-ID: <20011201155109.A45348@sunbay.com> References: <200111301040.fAUAeS519148@freefall.freebsd.org> <20011130205718.A275@nebula.noos.fr> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20011130205718.A275@nebula.noos.fr> User-Agent: Mutt/1.3.23i Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On Fri, Nov 30, 2001 at 08:57:18PM +0100, Maxime Henrion wrote: > Ruslan Ermilov wrote: > > ru 2001/11/30 02:40:28 PST > > > > Modified files: > > sys/netinet icmp_var.h ip_icmp.c ip_input.c ip_var.h > > usr.bin/netstat inet.c > [...] > > This commit appears to break the kernel. In some cases, icmp_reflect() > calls icmp_send() with a NULL pointer for the struct route * parameter. > icmp_send() then pass it to ip_output() which dereferences it. > > I've had several panics like this at boot, and also got some lockups for > reasons I ignore. :-) > > It seems to me that in the icmp_reflect() code it is intentional to call > icmp_send() with a NULL pointer, so I did a little patch that use a > struct route bzero'ed in that case in icmp_send() as it was in revision > 1.63. It probably isn't a correct patch but at least it solved the > problem. > Fixed in ip_output,v 1.143. Sorry. Cheers, -- Ruslan Ermilov Oracle Developer/DBA, ru@sunbay.com Sunbay Software AG, ru@FreeBSD.org FreeBSD committer, +380.652.512.251 Simferopol, Ukraine http://www.FreeBSD.org The Power To Serve http://www.oracle.com Enabling The Information Age To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message