diff options
author | Al Viro <viro@zeniv.linux.org.uk> | 2006-05-27 00:19:54 -0400 |
---|---|---|
committer | Al Viro <viro@zeniv.linux.org.uk> | 2006-05-27 02:26:32 -0400 |
commit | c7c0d0a10f6e4cea95ed84adcdd37948cac09b85 (patch) | |
tree | 4af26a97e8561766a9b0da9a29926ae645dc0468 /drivers/char/rio | |
parent | 38d0d001b973d8c72c18524cc25ef39db85c66cd (diff) |
[PATCH] bogus order of copy_from_user() arguments
... aka "somebody forgot to swap arguments when converting from copyin()"
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'drivers/char/rio')
-rw-r--r-- | drivers/char/rio/rioboot.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/char/rio/rioboot.c b/drivers/char/rio/rioboot.c index b9abbd0e7ec2..eca2b95343e2 100644 --- a/drivers/char/rio/rioboot.c +++ b/drivers/char/rio/rioboot.c | |||
@@ -280,7 +280,7 @@ int RIOBootCodeHOST(struct rio_info *p, struct DownLoad *rbp) | |||
280 | func_exit(); | 280 | func_exit(); |
281 | return -ENOMEM; | 281 | return -ENOMEM; |
282 | } | 282 | } |
283 | if (copy_from_user(rbp->DataP, DownCode, rbp->Count)) { | 283 | if (copy_from_user(DownCode, rbp->DataP, rbp->Count)) { |
284 | kfree(DownCode); | 284 | kfree(DownCode); |
285 | p->RIOError.Error = COPYIN_FAILED; | 285 | p->RIOError.Error = COPYIN_FAILED; |
286 | func_exit(); | 286 | func_exit(); |