Em Mon, 29 Jan 2018 11:47:42 +0200 Sakari Ailus <sakari.ailus@xxxxxx> escreveu: > > + compat_caddr_t p; > > + > > if (copy_to_user(&up->w, &kp->w, sizeof(kp->w)) || > > put_user(kp->field, &up->field) || > > put_user(kp->chromakey, &up->chromakey) || > > put_user(kp->clipcount, &up->clipcount) || > > put_user(kp->global_alpha, &up->global_alpha)) > > return -EFAULT; > > One more newline here? Why? A new line here would be weird. Cheers, Mauro