Commit 5842e51f authored by Al Viro's avatar Al Viro

[PATCH] pass mode instead of file to sg_io()

Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 86d434de
...@@ -190,12 +190,11 @@ void blk_set_cmd_filter_defaults(struct blk_cmd_filter *filter) ...@@ -190,12 +190,11 @@ void blk_set_cmd_filter_defaults(struct blk_cmd_filter *filter)
EXPORT_SYMBOL_GPL(blk_set_cmd_filter_defaults); EXPORT_SYMBOL_GPL(blk_set_cmd_filter_defaults);
static int blk_fill_sghdr_rq(struct request_queue *q, struct request *rq, static int blk_fill_sghdr_rq(struct request_queue *q, struct request *rq,
struct sg_io_hdr *hdr, struct file *file) struct sg_io_hdr *hdr, fmode_t mode)
{ {
if (copy_from_user(rq->cmd, hdr->cmdp, hdr->cmd_len)) if (copy_from_user(rq->cmd, hdr->cmdp, hdr->cmd_len))
return -EFAULT; return -EFAULT;
if (blk_verify_command(&q->cmd_filter, rq->cmd, if (blk_verify_command(&q->cmd_filter, rq->cmd, mode & FMODE_WRITE))
file->f_mode & FMODE_WRITE))
return -EPERM; return -EPERM;
/* /*
...@@ -260,8 +259,8 @@ static int blk_complete_sghdr_rq(struct request *rq, struct sg_io_hdr *hdr, ...@@ -260,8 +259,8 @@ static int blk_complete_sghdr_rq(struct request *rq, struct sg_io_hdr *hdr,
return r; return r;
} }
static int sg_io(struct file *file, struct request_queue *q, static int sg_io(struct request_queue *q, struct gendisk *bd_disk,
struct gendisk *bd_disk, struct sg_io_hdr *hdr) struct sg_io_hdr *hdr, fmode_t mode)
{ {
unsigned long start_time; unsigned long start_time;
int writing = 0, ret = 0; int writing = 0, ret = 0;
...@@ -293,7 +292,7 @@ static int sg_io(struct file *file, struct request_queue *q, ...@@ -293,7 +292,7 @@ static int sg_io(struct file *file, struct request_queue *q,
if (!rq) if (!rq)
return -ENOMEM; return -ENOMEM;
if (blk_fill_sghdr_rq(q, rq, hdr, file)) { if (blk_fill_sghdr_rq(q, rq, hdr, mode)) {
blk_put_request(rq); blk_put_request(rq);
return -EFAULT; return -EFAULT;
} }
...@@ -565,7 +564,7 @@ int scsi_cmd_ioctl(struct file *file, struct request_queue *q, ...@@ -565,7 +564,7 @@ int scsi_cmd_ioctl(struct file *file, struct request_queue *q,
err = -EFAULT; err = -EFAULT;
if (copy_from_user(&hdr, arg, sizeof(hdr))) if (copy_from_user(&hdr, arg, sizeof(hdr)))
break; break;
err = sg_io(file, q, bd_disk, &hdr); err = sg_io(q, bd_disk, &hdr, file ? file->f_mode : 0);
if (err == -EFAULT) if (err == -EFAULT)
break; break;
...@@ -613,7 +612,7 @@ int scsi_cmd_ioctl(struct file *file, struct request_queue *q, ...@@ -613,7 +612,7 @@ int scsi_cmd_ioctl(struct file *file, struct request_queue *q,
hdr.cmdp = ((struct cdrom_generic_command __user*) arg)->cmd; hdr.cmdp = ((struct cdrom_generic_command __user*) arg)->cmd;
hdr.cmd_len = sizeof(cgc.cmd); hdr.cmd_len = sizeof(cgc.cmd);
err = sg_io(file, q, bd_disk, &hdr); err = sg_io(q, bd_disk, &hdr, file ? file->f_mode : 0);
if (err == -EFAULT) if (err == -EFAULT)
break; break;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment