diff options
| author |   Kulikov Vasiliy <segooon@gmail.com> | 2010-08-03 12:52:55 +0200 |
|---|---|---|
| committer |   Jens Axboe <jaxboe@fusionio.com> | 2010-08-07 18:53:03 +0200 |
| commit | f6c4c8e19a087dae7dc651ccbd1ff8b843eedee2 (patch) | |
| tree | 49ef8d3044def969cd6bac6a3ef9e09d19a18803 /drivers/block/cpqarray.c | |
| parent | writeback: remove wb in get_next_work_item (diff) | |
| download | linux-dev-f6c4c8e19a087dae7dc651ccbd1ff8b843eedee2.tar.xz linux-dev-f6c4c8e19a087dae7dc651ccbd1ff8b843eedee2.zip | |
cpqarray: check put_user() result
put_user() may fail, if so return -EFAULT.
Signed-off-by: Kulikov Vasiliy <segooon@gmail.com>
Acked-by: Mike Miller <mike.miller@hp.com>
Signed-off-by: Jens Axboe <jaxboe@fusionio.com>
Diffstat (limited to 'drivers/block/cpqarray.c')
| -rw-r--r-- | drivers/block/cpqarray.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/block/cpqarray.c b/drivers/block/cpqarray.c index 9473215956f9..d53b0291c44b 100644 --- a/drivers/block/cpqarray.c +++ b/drivers/block/cpqarray.c @@ -1179,7 +1179,8 @@ out_passthru: return error; case IDAGETCTLRSIG: if (!arg) return -EINVAL; - put_user(host->ctlr_sig, (int __user *)arg); + if (put_user(host->ctlr_sig, (int __user *)arg)) + return -EFAULT; return 0; case IDAREVALIDATEVOLS: if (MINOR(bdev->bd_dev) != 0) @@ -1187,7 +1188,8 @@ out_passthru: return revalidate_allvol(host); case IDADRIVERVERSION: if (!arg) return -EINVAL; - put_user(DRIVER_VERSION, (unsigned long __user *)arg); + if (put_user(DRIVER_VERSION, (unsigned long __user *)arg)) + return -EFAULT; return 0; case IDAGETPCIINFO: { |