diff options
| author |   luoqing <luoqing@kylinos.cn> | 2025-02-08 09:35:39 +0800 |
|---|---|---|
| committer |   Andi Shyti <andi.shyti@linux.intel.com> | 2025-02-11 00:26:02 +0100 |
| commit | fc2ef5b6e1ab9717b1a0b588a39b410864091fa9 (patch) | |
| tree | 11fd7c4490f5ebcf5050eb989d93a51baa2933ea | |
| parent | drm/i915/selftests: avoid using uninitialized context (diff) | |
| download | wireguard-linux-fc2ef5b6e1ab9717b1a0b588a39b410864091fa9.tar.xz wireguard-linux-fc2ef5b6e1ab9717b1a0b588a39b410864091fa9.zip | |
selftests: i915: Use struct_size() helper in kmalloc()
Make use of the struct_size() helper instead of an open-coded version,
in order to avoid any potential type mistakes or integer overflows that,
in the worst scenario, could lead to heap overflows.
Signed-off-by: luoqing <luoqing@kylinos.cn>
Reviewed-by: Andi Shyti <andi.shyti@linux.intel.com>
Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250208013539.3586855-1-l1138897701@163.com
| -rw-r--r-- | drivers/gpu/drm/i915/gem/selftests/mock_dmabuf.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/i915/gem/selftests/mock_dmabuf.c b/drivers/gpu/drm/i915/gem/selftests/mock_dmabuf.c index 075657018739..5cd58e0f0dcf 100644 --- a/drivers/gpu/drm/i915/gem/selftests/mock_dmabuf.c +++ b/drivers/gpu/drm/i915/gem/selftests/mock_dmabuf.c @@ -103,8 +103,7 @@ static struct dma_buf *mock_dmabuf(int npages) struct dma_buf *dmabuf; int i; - mock = kmalloc(sizeof(*mock) + npages * sizeof(struct page *), - GFP_KERNEL); + mock = kmalloc(struct_size(mock, pages, npages), GFP_KERNEL); if (!mock) return ERR_PTR(-ENOMEM); |