mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-27 04:47:05 +00:00
drm_lease.c: copy user-array safely
Currently, there is no overflow-check with memdup_user(). Use the new function memdup_array_user() instead of memdup_user() for duplicating the user-space array safely. Suggested-by: David Airlie <airlied@redhat.com> Signed-off-by: Philipp Stanner <pstanner@redhat.com> Reviewed-by: Kees Cook <keescook@chromium.org> Reviewed-by: Zack Rusin <zackr@vmware.com> Signed-off-by: Dave Airlie <airlied@redhat.com> Link: https://patchwork.freedesktop.org/patch/msgid/20230920123612.16914-6-pstanner@redhat.com
This commit is contained in:
parent
ca0776571d
commit
f37d63e219
1 changed files with 2 additions and 2 deletions
|
@ -510,8 +510,8 @@ int drm_mode_create_lease_ioctl(struct drm_device *dev,
|
|||
/* Handle leased objects, if any */
|
||||
idr_init(&leases);
|
||||
if (object_count != 0) {
|
||||
object_ids = memdup_user(u64_to_user_ptr(cl->object_ids),
|
||||
array_size(object_count, sizeof(__u32)));
|
||||
object_ids = memdup_array_user(u64_to_user_ptr(cl->object_ids),
|
||||
object_count, sizeof(__u32));
|
||||
if (IS_ERR(object_ids)) {
|
||||
ret = PTR_ERR(object_ids);
|
||||
idr_destroy(&leases);
|
||||
|
|
Loading…
Reference in a new issue