]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
drm/amdgpu: reject non-user addresses early in GEM_USERPTR ioctl
authorAmir Shetaia <Amir.Shetaia@amd.com>
Thu, 7 May 2026 17:24:55 +0000 (13:24 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 19 May 2026 16:08:47 +0000 (12:08 -0400)
commit2c34c7b88ba4f04b3a862fac879bfd8567f06541
tree39a337ef5055f1105ce3c4bfcb4e3fee3575cf12
parentb6074630a461b1322a814988779005cbc43612ea
drm/amdgpu: reject non-user addresses early in GEM_USERPTR ioctl

amdgpu_gem_userptr_ioctl() currently accepts any value of args->addr
and only discovers an out-of-range pointer much later, inside
amdgpu_gem_object_create() and the HMM mirror registration path.
Userspace can drive that path with kernel-side virtual addresses;
the get_user_pages() layer rejects them, but only after the driver
has already allocated a GEM object and started wiring up notifier
state that then has to be torn down on failure.

Add an access_ok() guard at the top of the ioctl, right after the
existing page-alignment check and before flag validation, so any
address that does not lie within the calling task's user address
range is rejected with -EFAULT before any allocation occurs. No
legitimate ROCm/HSA userspace passes kernel-mode pointers through
this interface, so this is defense-in-depth rather than a behaviour
change for valid callers; -EFAULT matches the convention already
used by other uaccess-style rejections in the kernel.

Also add an explicit #include <linux/uaccess.h>; access_ok() is
otherwise only available transitively through other headers in
this translation unit.

Signed-off-by: Amir Shetaia <Amir.Shetaia@amd.com>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
(cherry picked from commit 7a076df36397d780d7e4fb595287b4980451a7f5)
drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c