From owner-svn-src-all@FreeBSD.ORG Fri Jul 31 04:37:53 2009 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 5D3E6106566B; Fri, 31 Jul 2009 04:37:53 +0000 (UTC) (envelope-from oliver.pntr@gmail.com) Received: from fg-out-1718.google.com (fg-out-1718.google.com [72.14.220.154]) by mx1.freebsd.org (Postfix) with ESMTP id 642CA8FC0C; Fri, 31 Jul 2009 04:37:52 +0000 (UTC) (envelope-from oliver.pntr@gmail.com) Received: by fg-out-1718.google.com with SMTP id l26so27811fgb.12 for ; Thu, 30 Jul 2009 21:37:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:cc:content-type; bh=/iOuqgYHckQLqV60RO68nwHTU7Wi6wquWXBAzj9MtXk=; b=I/50Wo61k4xxdiIlejM8HoOkHhJ+PsqpN90zcpW3kSWFy/8yRWKNeq0jyp/ai6Y7j7 N7JL2iqIRZ8h64Q55HyGTA+gaeq/W2918yvinguqtoCvpLK+FN+JblH8u/FeYdgcngz5 koHbQYSGsCr+iOD0I+EgP/WzSMbDoEnZF40G8= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; b=hoG2eGjuXkgW+x0l6xlc2Rjky5O1pfRgPpR9sfBZbEMBOI156seE4Ho+H6wywC2Gct Op0GSLmO9VNj9q4y+F8Fa2Gx7rGx+FO9SXBTF8aaJnGg6gOtQlN0v37k/GYq9pxvZA2H 65oG1a5+F37EhvEgBr1FTnCdzfe1YwCssbn/0= MIME-Version: 1.0 Received: by 10.86.87.1 with SMTP id k1mr675934fgb.74.1249013666728; Thu, 30 Jul 2009 21:14:26 -0700 (PDT) In-Reply-To: <200907301319.n6UDJC6B014743@svn.freebsd.org> References: <200907301319.n6UDJC6B014743@svn.freebsd.org> Date: Fri, 31 Jul 2009 06:14:26 +0200 Message-ID: <6101e8c40907302114l46e6a0f2wa783384a80504a0@mail.gmail.com> From: Oliver Pinter To: Alexander Motin Content-Type: multipart/mixed; boundary=000e0cd2a030eccac7046ff8a7ed Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-7@freebsd.org Subject: Re: svn commit: r195973 - in stable/7/sys: . contrib/pf dev/ata X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 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: Fri, 31 Jul 2009 04:37:54 -0000 --000e0cd2a030eccac7046ff8a7ed Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit fix complie error, tested, booted, used On 7/30/09, Alexander Motin wrote: > Author: mav > Date: Thu Jul 30 13:19:12 2009 > New Revision: 195973 > URL: http://svn.freebsd.org/changeset/base/195973 > > Log: > MFC rev. 195724: > > Limit IOCATAREQUEST ioctl data size to controller's maximum I/O size. > It fixes kernel panic when requested size is too large (0xffffffff). > > PR: kern/136726 > > Modified: > stable/7/sys/ (props changed) > stable/7/sys/contrib/pf/ (props changed) > stable/7/sys/dev/ata/ata-all.c > > Modified: stable/7/sys/dev/ata/ata-all.c > ============================================================================== > --- stable/7/sys/dev/ata/ata-all.c Thu Jul 30 12:41:19 2009 (r195972) > +++ stable/7/sys/dev/ata/ata-all.c Thu Jul 30 13:19:12 2009 (r195973) > @@ -440,6 +440,7 @@ int > ata_device_ioctl(device_t dev, u_long cmd, caddr_t data) > { > struct ata_device *atadev = device_get_softc(dev); > + struct ata_channel *ch = device_get_softc(device_get_parent(dev)); > struct ata_ioc_request *ioc_request = (struct ata_ioc_request *)data; > struct ata_params *params = (struct ata_params *)data; > int *mode = (int *)data; > @@ -449,6 +450,10 @@ ata_device_ioctl(device_t dev, u_long cm > > switch (cmd) { > case IOCATAREQUEST: > + if (ioc_request->count > > + (ch->dma.max_iosize ? ch->dma.max_iosize : DFLTPHYS)) { > + return (EFBIG); > + } > if (!(buf = malloc(ioc_request->count, M_ATA, M_NOWAIT))) { > return ENOMEM; > } > _______________________________________________ > svn-src-stable-7@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/svn-src-stable-7 > To unsubscribe, send any mail to "svn-src-stable-7-unsubscribe@freebsd.org" > --000e0cd2a030eccac7046ff8a7ed Content-Type: text/x-diff; charset=US-ASCII; name="ata_compile_fix.diff" Content-Disposition: attachment; filename="ata_compile_fix.diff" Content-Transfer-Encoding: base64 X-Attachment-Id: file0 Y29tbWl0IDVmNDkyMGI5MWU5MTUxNDA5OWQxYTMyMWU3ZWQ4MjcwMDc2MDIxYjUKQXV0aG9yOiBv cHYgPG9wdkBvbGl2ZXJwLnRldGVueS5ibWUuaHU+CkRhdGU6ICAgRnJpIEp1bCAzMSAwNTo1Njoy MyAyMDA5ICswMjAwCgogICAgYXRhIGNvbXBsaWUgZml4IC0gd2FzIHN2biBjb21taXQ6IHIxOTU5 NzMgZml4CgogICAgL3Vzci9zcmMvc3lzL2Rldi9hdGEvYXRhLWFsbC5jOiBJbiBmdW5jdGlvbiAn YXRhX2RldmljZV9pb2N0bCc6CiAgICAvdXNyL3NyYy9zeXMvZGV2L2F0YS9hdGEtYWxsLmM6NDU0 OiBlcnJvcjogcmVxdWVzdCBmb3IgbWVtYmVyICdtYXhfaW9zaXplJwogICAgaW4gc29tZXRoaW5n IG5vdCBhIHN0cnVjdHVyZSBvciB1bmlvbgoKICAgIFNpZ25lZC1vZmYtYnk6IE9saXZlciBQaW50 ZXIgPG9saXZlci5wbnRyQGdtYWlsLmNvbT4KCgpkaWZmIC0tZ2l0IGEvc3lzL2Rldi9hdGEvYXRh LWFsbC5jIGIvc3lzL2Rldi9hdGEvYXRhLWFsbC5jCmluZGV4IGM1OGU2NzUuLjc5ZTBiYzQgMTAw NjQ0Ci0tLSBhL3N5cy9kZXYvYXRhL2F0YS1hbGwuYworKysgYi9zeXMvZGV2L2F0YS9hdGEtYWxs LmMKQEAgLTQ1MSw3ICs0NTEsNyBAQCBhdGFfZGV2aWNlX2lvY3RsKGRldmljZV90IGRldiwgdV9s b25nIGNtZCwgY2FkZHJfdCBkYXRhKQogICAgIHN3aXRjaCAoY21kKSB7CiAgICAgY2FzZSBJT0NB VEFSRVFVRVNUOgogCWlmIChpb2NfcmVxdWVzdC0+Y291bnQgPgotCSAgICAoY2gtPmRtYS5tYXhf aW9zaXplID8gY2gtPmRtYS5tYXhfaW9zaXplIDogREZMVFBIWVMpKSB7CisJICAgIChjaC0+ZG1h LT5tYXhfaW9zaXplID8gY2gtPmRtYS0+bWF4X2lvc2l6ZSA6IERGTFRQSFlTKSkgewogCQlyZXR1 cm4gKEVGQklHKTsKIAl9CiAJaWYgKCEoYnVmID0gbWFsbG9jKGlvY19yZXF1ZXN0LT5jb3VudCwg TV9BVEEsIE1fTk9XQUlUKSkpIHsK --000e0cd2a030eccac7046ff8a7ed--