On Tue, Feb 25, 2025 at 05:47:06PM -0800, Harish Chegondi wrote:
> +static int xe_eu_stall_user_ext_set_property(struct xe_device *xe, u64 
> extension,
> +                                          struct eu_stall_open_properties 
> *props)
> +{
> +     u64 __user *address = u64_to_user_ptr(extension);
> +     struct drm_xe_ext_set_property ext;
> +     int err;
> +     u32 idx;
> +
> +     err = __copy_from_user(&ext, address, sizeof(ext));

Why is this safe? (i.e. why is it not copy_from_user()?) I see no
access_ok() check anywhere in the ioctl call path to this function.

-- 
Kees Cook

Reply via email to