Skip to content

Commit

Permalink
[PATCH] bogus order of copy_from_user() arguments
Browse files Browse the repository at this point in the history
... aka "somebody forgot to swap arguments when converting from copyin()"

Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
  • Loading branch information
Al Viro committed May 27, 2006
1 parent 38d0d00 commit c7c0d0a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/char/rio/rioboot.c
Original file line number Diff line number Diff line change
Expand Up @@ -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();
Expand Down

0 comments on commit c7c0d0a

Please sign in to comment.