From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx0b-001b2d01.pphosted.com ([148.163.158.5]:58939 "EHLO mx0a-001b2d01.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752680AbdHHOjU (ORCPT ); Tue, 8 Aug 2017 10:39:20 -0400 Received: from pps.filterd (m0098419.ppops.net [127.0.0.1]) by mx0b-001b2d01.pphosted.com (8.16.0.21/8.16.0.21) with SMTP id v78EXcSG099501 for ; Tue, 8 Aug 2017 10:39:19 -0400 Received: from e11.ny.us.ibm.com (e11.ny.us.ibm.com [129.33.205.201]) by mx0b-001b2d01.pphosted.com with ESMTP id 2c7b3d1u5u-1 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=NOT) for ; Tue, 08 Aug 2017 10:39:19 -0400 Received: from localhost by e11.ny.us.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 8 Aug 2017 10:39:18 -0400 Subject: Re: Patch "scsi: sg: fix SG_DXFER_FROM_DEV transfers" has been added to the 4.12-stable tree From: James Bottomley To: Johannes Thumshirn , Chris Clayton , gregkh@linuxfoundation.org Cc: dgilbert@interlog.com, hare@suse.com, martin.petersen@oracle.com, stable@vger.kernel.org, stable-commits@vger.kernel.org Date: Tue, 08 Aug 2017 07:39:12 -0700 In-Reply-To: <20170808121935.GH3926@linux-x5ow.site> References: <1502137958231142@kroah.com> <7f2007f7-1952-782e-b694-ed07b57008bb@googlemail.com> <20170808121935.GH3926@linux-x5ow.site> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Message-Id: <1502203152.2736.3.camel@linux.vnet.ibm.com> Sender: stable-owner@vger.kernel.org List-ID: On Tue, 2017-08-08 at 14:20 +0200, Johannes Thumshirn wrote: > On Tue, Aug 08, 2017 at 12:42:17PM +0100, Chris Clayton wrote: > > > > On 07/08/17 21:32, gregkh@linuxfoundation.org wrote: > > > > > > > > > This is a note to let you know that I've just added the patch > > > titled > > > > > >     scsi: sg: fix SG_DXFER_FROM_DEV transfers > > > > > > to the 4.12-stable tree which can be found at: > > >     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-q > > > ueue.git;a=summary > > > > > > The filename of the patch is: > > >      scsi-sg-fix-sg_dxfer_from_dev-transfers.patch > > > and it can be found in the queue-4.12 subdirectory. > > > > > > If you, or anyone else, feels it should not be added to the > > > stable tree, > > > please let know about it. > > > > > > > Johannes' commit message says that the transfer must have a length > > bigger than 0, so the code should return false if the > > length is less than or equal to 0, but the test is for less than 0. > > > > But in any case, there's another patch that removes all this > > sg_is_valid_dxfer() jiggery-pokery and replaces it with a > > simpler test. It hasn't reached Linus' tree yet but is, I believe, > > cc'd to stable. > > Yup, I hope James is sending it out for this weekend's -rc and then > it should be backported to stable instead. I'm not sure which patch you mean.  However, if you got one of my automated emails about it then it will be in today's batch for Linus. James > James any plans in doing otherwise? > Greg is this acceptible for you? > > Thanks, > Johannes