diff options
author | Tony Battersby <tonyb@cybernetics.com> | 2018-07-11 16:46:03 +0200 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2018-07-11 16:48:28 +0200 |
commit | 70dbcc2254fa2a9add74a122b9dac954c4736e01 (patch) | |
tree | d68a78ec09666e1e9e142a4bc587c18059674a4f /block | |
parent | loop: Add LOOP_SET_BLOCK_SIZE in compat ioctl (diff) | |
download | linux-70dbcc2254fa2a9add74a122b9dac954c4736e01.tar.xz linux-70dbcc2254fa2a9add74a122b9dac954c4736e01.zip |
bsg: fix bogus EINVAL on non-data commands
Fix a regression introduced in Linux kernel 4.17 where sending a SCSI
command that does not transfer data (such as TEST UNIT READY) via
/dev/bsg/* results in EINVAL.
Fixes: 17cb960f29c2 ("bsg: split handling of SCSI CDBs vs transport requeues")
Cc: <stable@vger.kernel.org> # 4.17+
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Tony Battersby <tonyb@cybernetics.com>
Signed-off-by: Jens Axboe <axboe@kernel.dk>
Diffstat (limited to 'block')
-rw-r--r-- | block/bsg.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/block/bsg.c b/block/bsg.c index 66602c489956..3da540faf673 100644 --- a/block/bsg.c +++ b/block/bsg.c @@ -267,8 +267,6 @@ bsg_map_hdr(struct request_queue *q, struct sg_io_v4 *hdr, fmode_t mode) } else if (hdr->din_xfer_len) { ret = blk_rq_map_user(q, rq, NULL, uptr64(hdr->din_xferp), hdr->din_xfer_len, GFP_KERNEL); - } else { - ret = blk_rq_map_user(q, rq, NULL, NULL, 0, GFP_KERNEL); } if (ret) |