aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu
diff options
context:
space:
mode:
authorQintaoShen <unSimple1993@163.com>2022-03-24 16:26:23 +0800
committerAlex Deucher <alexander.deucher@amd.com>2022-03-25 12:40:25 -0400
commitebbb7bb9e80305820dc2328a371c1b35679f2667 (patch)
tree0f793acb076ecbc42122c312b233b7cb3f936f3a /drivers/gpu
parentdrm/amdgpu/vcn: improve vcn dpg stop procedure (diff)
downloadlinux-dev-ebbb7bb9e80305820dc2328a371c1b35679f2667.tar.xz
linux-dev-ebbb7bb9e80305820dc2328a371c1b35679f2667.zip
drm/amdkfd: Check for potential null return of kmalloc_array()
As the kmalloc_array() may return null, the 'event_waiters[i].wait' would lead to null-pointer dereference. Therefore, it is better to check the return value of kmalloc_array() to avoid this confusion. Signed-off-by: QintaoShen <unSimple1993@163.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu')
-rw-r--r--drivers/gpu/drm/amd/amdkfd/kfd_events.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_events.c b/drivers/gpu/drm/amd/amdkfd/kfd_events.c
index deecccebe5b6..64f4a51cc880 100644
--- a/drivers/gpu/drm/amd/amdkfd/kfd_events.c
+++ b/drivers/gpu/drm/amd/amdkfd/kfd_events.c
@@ -749,6 +749,8 @@ static struct kfd_event_waiter *alloc_event_waiters(uint32_t num_events)
event_waiters = kmalloc_array(num_events,
sizeof(struct kfd_event_waiter),
GFP_KERNEL);
+ if (!event_waiters)
+ return NULL;
for (i = 0; (event_waiters) && (i < num_events) ; i++) {
init_wait(&event_waiters[i].wait);