---- in dvb-frontend.c: ---- if(cmd == FE_GET_PROPERTY) { tvps = (struct dtv_properties __user *)parg; dprintk("%s() properties.num = %d\n", __func__, tvps->num); dprintk("%s() properties.props = %p\n", __func__, tvps->props); ... if (copy_from_user(tvp, tvps->props, tvps->num * sizeof(struct dtv_property))) ---- > OK, > > thought I'll have never to care for it again. > > ENUM calls should never be W. > > Hit me for all I missed. > > Cheers, > Hermann you are not seeing the point of it it seems Documentation/ioctl-number.txt ---- If you are adding new ioctl's to the kernel, you should use the _IO macros defined in <linux/ioctl.h>: _IO an ioctl with no parameters _IOW an ioctl with write parameters (copy_from_user) _IOR an ioctl with read parameters (copy_to_user) _IOWR an ioctl with both write and read parameters. ---- copy from user is required in order to copy the keys for the requested elements into the kernel. copy to user is finally used to play them back. Markus -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html