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 | |
| parent | 38d0d001b973d8c72c18524cc25ef39db85c66cd (diff) | |
| download | op-kernel-dev-c7c0d0a10f6e4cea95ed84adcdd37948cac09b85.zip op-kernel-dev-c7c0d0a10f6e4cea95ed84adcdd37948cac09b85.tar.gz | |
[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>
| -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 b9abbd0..eca2b95 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) func_exit(); return -ENOMEM; } - if (copy_from_user(rbp->DataP, DownCode, rbp->Count)) { + if (copy_from_user(DownCode, rbp->DataP, rbp->Count)) { kfree(DownCode); p->RIOError.Error = COPYIN_FAILED; func_exit(); |
