> 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@xxxxxxxxx> > Acked-by: Harsha Priya <priya.harsha@xxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html