diff options
author | Mark Haverkamp <markh@osdl.org> | 2006-06-12 17:53:58 (GMT) |
---|---|---|
committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-06-12 19:47:55 (GMT) |
commit | 821499a8241019be5591fad5ab925759fbcd9f32 (patch) | |
tree | 2a68cbe5ad747412afa085fbca2fc56b19d10722 /drivers/scsi | |
parent | c5943d36aebb79589ac3983eec418bbeca897573 (diff) | |
download | linux-fsl-qoriq-821499a8241019be5591fad5ab925759fbcd9f32.tar.xz |
[SCSI] aacraid: Fix return code interpretation
Received from Mark Salyzyn
clear_user return is 0 for success, the code fragment is written to
assume that it is the count of the number of bytes zero'd.
Signed-off-by: Mark Haverkamp <markh@osdl.org>
Signed-off-by: James Bottomley <James.Bottomley@SteelEye.com>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/aacraid/linit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/aacraid/linit.c b/drivers/scsi/aacraid/linit.c index 9647dab..03fe2ab 100644 --- a/drivers/scsi/aacraid/linit.c +++ b/drivers/scsi/aacraid/linit.c @@ -564,7 +564,7 @@ static long aac_compat_do_ioctl(struct aac_dev *dev, unsigned cmd, unsigned long f = compat_alloc_user_space(sizeof(*f)); ret = 0; - if (clear_user(f, sizeof(*f)) != sizeof(*f)) + if (clear_user(f, sizeof(*f))) ret = -EFAULT; if (copy_in_user(f, (void __user *)arg, sizeof(struct fib_ioctl) - sizeof(u32))) ret = -EFAULT; |