Commit c9e20cb0 authored by Arunpravin Paneer Selvam's avatar Arunpravin Paneer Selvam Committed by Alex Deucher
Browse files

drm/amdgpu: Fix NULL ptr dereference issue for non userq fences 



Add the correct fences count variable [num_fences] in the fences
array iteration to handle the userq / non-userq fences.

v2:(Christian)
  - All fences in the array either come from some reservation object
    or drm_syncobj. If any of those are NULL then there is a bug
    somewhere else.

Signed-off-by: default avatarArunpravin Paneer Selvam <Arunpravin.PaneerSelvam@amd.com>
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 9ed335d9

drivers/gpu/drm/amd/amdgpu/amdgpu_userq_fence.c

+1 −1
Original line number Diff line number Diff line
@@ -817,7 +817,7 @@ int amdgpu_userq_wait_ioctl(struct drm_device *dev, void *data, 
			fences[num_fences++] = fence;

		}



		for (i = 0, cnt = 0; i < wait_info->num_fences; i++) {

		for (i = 0, cnt = 0; i < num_fences; i++) {

			struct amdgpu_userq_fence_driver *fence_drv;

			struct amdgpu_userq_fence *userq_fence;

			u32 index;