Commit c7c0d0a1 authored by Al Viro's avatar Al Viro
Browse files

[PATCH] bogus order of copy_from_user() arguments 



... aka "somebody forgot to swap arguments when converting from copyin()"

Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 38d0d001

drivers/char/rio/rioboot.c

+1 −1
Original line number Diff line number Diff line
@@ -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();