aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/include/linux/reset/ssh:/git@git.zx2c4.com
diff options
context:
space:
mode:
authorAmir Shetaia <Amir.Shetaia@amd.com>2026-05-07 13:24:55 -0400
committerAlex Deucher <alexander.deucher@amd.com>2026-05-19 12:08:47 -0400
commit2c34c7b88ba4f04b3a862fac879bfd8567f06541 (patch)
tree39a337ef5055f1105ce3c4bfcb4e3fee3575cf12 /include/linux/reset/ssh:/git@git.zx2c4.com
parentdrm/amdgpu/vpe: Force collaborate sync after TRAP (diff)
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)
Diffstat (limited to 'include/linux/reset/ssh:/git@git.zx2c4.com')
0 files changed, 0 insertions, 0 deletions