diff options
author | Jesper Juhl <jesper.juhl@gmail.com> | 2006-09-15 08:43:11 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-09-23 12:53:09 -0400 |
commit | 2d2f8d59b14bec6c745e219a350ac51d9e00673f (patch) | |
tree | 0e40289425e854c0b7f6e899600f2abd9a6df6bd /drivers/scsi | |
parent | 5fcda4224529c4e550c917668d5e96c1d3e7039b (diff) |
[SCSI] megaraid: Make megaraid_ioctl() check copy_to_user() return value
Check copy_to_user() return value in drivers/scsi/megaraid.c::megadev_ioctl()
This gets rid of this little warning:
drivers/scsi/megaraid.c:3661: warning: ignoring return value of 'copy_to_user', declared with attribute warn_unused_result
Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
Acked-by: "Ju, Seokmann" <Seokmann.Ju@lsil.com>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/megaraid.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/scsi/megaraid.c b/drivers/scsi/megaraid.c index ccb0055ac73a..b87bef69ba0f 100644 --- a/drivers/scsi/megaraid.c +++ b/drivers/scsi/megaraid.c | |||
@@ -3656,8 +3656,9 @@ megadev_ioctl(struct inode *inode, struct file *filep, unsigned int cmd, | |||
3656 | * Send the request sense data also, irrespective of | 3656 | * Send the request sense data also, irrespective of |
3657 | * whether the user has asked for it or not. | 3657 | * whether the user has asked for it or not. |
3658 | */ | 3658 | */ |
3659 | copy_to_user(upthru->reqsensearea, | 3659 | if (copy_to_user(upthru->reqsensearea, |
3660 | pthru->reqsensearea, 14); | 3660 | pthru->reqsensearea, 14)) |
3661 | rval = -EFAULT; | ||
3661 | 3662 | ||
3662 | freemem_and_return: | 3663 | freemem_and_return: |
3663 | if( pthru->dataxferlen ) { | 3664 | if( pthru->dataxferlen ) { |