From owner-cvs-all Wed Feb 7 19:19:10 2001 Delivered-To: cvs-all@freebsd.org Received: from aslan.scsiguy.com (aslan.scsiguy.com [63.229.232.106]) by hub.freebsd.org (Postfix) with ESMTP id E9C3837B69E; Wed, 7 Feb 2001 19:18:47 -0800 (PST) Received: from scsiguy.com (localhost [127.0.0.1]) by aslan.scsiguy.com (8.11.2/8.9.3) with ESMTP id f183IhO03934; Wed, 7 Feb 2001 20:18:47 -0700 (MST) (envelope-from gibbs@scsiguy.com) Message-Id: <200102080318.f183IhO03934@aslan.scsiguy.com> X-Mailer: exmh version 2.2 06/23/2000 with nmh-1.0.4 To: John Baldwin Cc: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: Re: cvs commit: src/sys/dev/advansys adv_isa.c advansys.c In-Reply-To: Your message of "Wed, 07 Feb 2001 16:00:15 PST." <200102080000.f1800Fn42789@freefall.freebsd.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Date: Wed, 07 Feb 2001 20:18:42 -0700 From: "Justin T. Gibbs" Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG >jhb 2001/02/07 16:00:15 PST > > Modified files: > sys/dev/advansys adv_isa.c advansys.c > Log: > - Check if the ccb_infos in the softc is NULL before trying to free it in > adv_free() as the ISA probe routine doesn't malloc() ccb_infos but does > call adv_free(). > - Release the ISA-only overrun DMA tags, bufs, and maps if the probe fails. It would have been nice of you to contact the driver author for review. -- Justin To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message