[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>

diff --git a/drivers/char/rio/rioboot.c b/drivers/char/rio/rioboot.c
index b9abbd0e7ec26df5a4b97f422164ab42ccbf6c2b..eca2b95343e2c3f63a9de30dfa705a8d412e485a 100644 (file)
--- 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();