[PATCH] scsi: sg: fix SG_DXFER_FROM_DEV transfers

[PATCH] scsi: sg: fix SG_DXFER_FROM_DEV transfers

[Johannes Thumshirn]

(commit 68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47 upstream)

SG_DXFER_FROM_DEV transfers do not necessarily have a dxferp as we set
it to NULL for the old sg_io read/write interface, but must have a
length bigger than 0. This fixes a regression introduced by commit
28676d869bbb ("scsi: sg: check for valid direction before starting the
request")

Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
Fixes: 28676d869bbb ("scsi: sg: check for valid direction before starting the request")
Reported-by: Chris Clayton <chris2553@googlemail.com>
Tested-by: Chris Clayton <chris2553@googlemail.com>
Cc: Douglas Gilbert <dgilbert@interlog.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Tested-by: Chris Clayton <chris2553@googlemail.com>
Acked-by: Douglas Gilbert <dgilbert@interlog.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
---
 drivers/scsi/sg.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/drivers/scsi/sg.c b/drivers/scsi/sg.c
index 21225d62b0c1..1e82d4128a84 100644
--- a/drivers/scsi/sg.c
+++ b/drivers/scsi/sg.c
@@ -758,8 +758,11 @@ static bool sg_is_valid_dxfer(sg_io_hdr_t *hp)
 		if (hp->dxferp || hp->dxfer_len > 0)
 			return false;
 		return true;
-	case SG_DXFER_TO_DEV:
 	case SG_DXFER_FROM_DEV:
+		if (hp->dxfer_len < 0)
+			return false;
+		return true;
+	case SG_DXFER_TO_DEV:
 	case SG_DXFER_TO_FROM_DEV:
 		if (!hp->dxferp || hp->dxfer_len == 0)
 			return false;
-- 
2.12.3

Re: [PATCH] scsi: sg: fix SG_DXFER_FROM_DEV transfers

[Greg Kroah-Hartman]

On Tue, Jul 18, 2017 at 09:42:21AM +0200, Johannes Thumshirn wrote:
> (commit 68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47 upstream)
> 
> SG_DXFER_FROM_DEV transfers do not necessarily have a dxferp as we set
> it to NULL for the old sg_io read/write interface, but must have a
> length bigger than 0. This fixes a regression introduced by commit
> 28676d869bbb ("scsi: sg: check for valid direction before starting the
> request")
> 
> Signed-off-by: Johannes Thumshirn <jthumshirn@suse.de>
> Fixes: 28676d869bbb ("scsi: sg: check for valid direction before starting the request")
> Reported-by: Chris Clayton <chris2553@googlemail.com>
> Tested-by: Chris Clayton <chris2553@googlemail.com>
> Cc: Douglas Gilbert <dgilbert@interlog.com>
> Reviewed-by: Hannes Reinecke <hare@suse.com>
> Tested-by: Chris Clayton <chris2553@googlemail.com>
> Acked-by: Douglas Gilbert <dgilbert@interlog.com>
> Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
> ---
>  drivers/scsi/sg.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)

What stable tree(s) do you want this to be applied to?

thanks,

greg k-h

Re: [PATCH] scsi: sg: fix SG_DXFER_FROM_DEV transfers

[Johannes Thumshirn]

On Tue, Jul 18, 2017 at 09:50:07AM +0200, Greg KH wrote:
> What stable tree(s) do you want this to be applied to?

I only have reports for 4.12 and this is where the broken commit went in. So I
guess 4.12 should be sufficient.

Thanks,
	Johannes

-- 
Johannes Thumshirn                                          Storage
jthumshirn@suse.de                                +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N�rnberg
GF: Felix Imend�rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N�rnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850

Re: [PATCH] scsi: sg: fix SG_DXFER_FROM_DEV transfers

[Greg Kroah-Hartman]

On Tue, Jul 18, 2017 at 09:42:21AM +0200, Johannes Thumshirn wrote:
> (commit 68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47 upstream)

There is no such commit in Linus's tree :(

Where did this come from?

thanks,

greg k-h

Re: [PATCH] scsi: sg: fix SG_DXFER_FROM_DEV transfers

[Martin K. Petersen]

Greg,

> On Tue, Jul 18, 2017 at 09:42:21AM +0200, Johannes Thumshirn wrote:
>> (commit 68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47 upstream)
>
> There is no such commit in Linus's tree :(

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/scsi/sg.c?id=68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47

-- 
Martin K. Petersen	Oracle Linux Engineering

Re: [PATCH] scsi: sg: fix SG_DXFER_FROM_DEV transfers

[Greg Kroah-Hartman]

On Tue, Jul 18, 2017 at 11:54:31AM -0400, Martin K. Petersen wrote:
> 
> Greg,
> 
> > On Tue, Jul 18, 2017 at 09:42:21AM +0200, Johannes Thumshirn wrote:
> >> (commit 68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47 upstream)
> >
> > There is no such commit in Linus's tree :(
> 
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/scsi/sg.c?id=68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47

{sigh} I hadn't updated my local tree.

As this isn't in a -rc kernel yet, is it ok for me to wait until it
shows up in one before adding it to a stable tree?

And why wasn't it marked with a "cc: stable" in it so this would all
happen automatically?

thanks,

greg k-h

Re: [PATCH] scsi: sg: fix SG_DXFER_FROM_DEV transfers

[Johannes Thumshirn]

On Tue, Jul 18, 2017 at 06:18:29PM +0200, Greg KH wrote:
> On Tue, Jul 18, 2017 at 11:54:31AM -0400, Martin K. Petersen wrote:
> > 
> > Greg,
> > 
> > > On Tue, Jul 18, 2017 at 09:42:21AM +0200, Johannes Thumshirn wrote:
> > >> (commit 68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47 upstream)
> > >
> > > There is no such commit in Linus's tree :(
> > 
> > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/scsi/sg.c?id=68c59fcea1f2c6a54c62aa896cc623c1b5bc9b47
> 
> {sigh} I hadn't updated my local tree.
> 
> As this isn't in a -rc kernel yet, is it ok for me to wait until it
> shows up in one before adding it to a stable tree?
> 
> And why wasn't it marked with a "cc: stable" in it so this would all
> happen automatically?

Because I forgot it. I'm sorry.

-- 
Johannes Thumshirn                                          Storage
jthumshirn@suse.de                                +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 N�rnberg
GF: Felix Imend�rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N�rnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850