projects
/
platform
/
kernel
/
linux-starfive.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
0f403eb
)
drm_lease.c: copy user-array safely
author
Philipp Stanner
<pstanner@redhat.com>
Wed, 20 Sep 2023 12:36:12 +0000
(14:36 +0200)
committer
Greg Kroah-Hartman
<gregkh@linuxfoundation.org>
Tue, 28 Nov 2023 17:19:40 +0000
(17:19 +0000)
[ Upstream commit
f37d63e219c39199a59b8b8a211412ff27192830
]
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
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/gpu/drm/drm_lease.c
patch
|
blob
|
history
diff --git
a/drivers/gpu/drm/drm_lease.c
b/drivers/gpu/drm/drm_lease.c
index
150fe15
..
94375c6
100644
(file)
--- a/
drivers/gpu/drm/drm_lease.c
+++ b/
drivers/gpu/drm/drm_lease.c
@@
-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);