diff options
author | Mark Haverkamp <markh@osdl.org> | 2006-06-12 13:53:58 -0400 |
---|---|---|
committer | James Bottomley <jejb@mulgrave.il.steeleye.com> | 2006-06-12 15:47:55 -0400 |
commit | 821499a8241019be5591fad5ab925759fbcd9f32 (patch) | |
tree | 2a68cbe5ad747412afa085fbca2fc56b19d10722 | |
parent | c5943d36aebb79589ac3983eec418bbeca897573 (diff) |
[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>
-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 9647dabc6612..03fe2abcc4fe 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 | |||
564 | 564 | ||
565 | f = compat_alloc_user_space(sizeof(*f)); | 565 | f = compat_alloc_user_space(sizeof(*f)); |
566 | ret = 0; | 566 | ret = 0; |
567 | if (clear_user(f, sizeof(*f)) != sizeof(*f)) | 567 | if (clear_user(f, sizeof(*f))) |
568 | ret = -EFAULT; | 568 | ret = -EFAULT; |
569 | if (copy_in_user(f, (void __user *)arg, sizeof(struct fib_ioctl) - sizeof(u32))) | 569 | if (copy_in_user(f, (void __user *)arg, sizeof(struct fib_ioctl) - sizeof(u32))) |
570 | ret = -EFAULT; | 570 | ret = -EFAULT; |