]> bbs.cooldavid.org Git - net-next-2.6.git/commit
Staging: sst: more copy_to_user() changes
authorDan Carpenter <error27@gmail.com>
Fri, 15 Oct 2010 19:10:44 +0000 (21:10 +0200)
committerGreg Kroah-Hartman <gregkh@suse.de>
Fri, 15 Oct 2010 22:42:26 +0000 (15:42 -0700)
commit388b2b97ba03242145227821c77791e713d6057b
treefc6f8ebfee86ea3e01316ed2e3eb77b143b14e3e
parent9f407840bb888e683fa844b1b80eec54ced84653
Staging: sst: more copy_to_user() changes

Vinod wanted the copy_to_user() calls in this format:
if (copy_to_user())
retval = -EFAULT;
instead of this:
retval = copy_to_user();
if (retval)
retval = -EFAULt;

I've done that for the whole intel_sst_app_interface.c file.  In the
process I noticed a couple more places that returned -EBUSY or -EAGAIN
instead of -EFAULT.

Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/intel_sst/intel_sst_app_interface.c