diff options
author | Colin Ian King <colin.king@canonical.com> | 2016-09-20 11:49:31 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-09-22 10:24:17 -0400 |
commit | 664a08bb906bd946a4c3963d5f92518e5ff19bd8 (patch) | |
tree | ae1b506079a1561eba9dc1ed157c2a46dc3295e3 /drivers/gpu/drm/amd/amdgpu | |
parent | 4d446656fabb308ae2171e95a624a0367dcd34d2 (diff) |
drm/amdgpu: avoid out of bounds access on array interrupt_status_offsets
The check for an out of bound index into array interrupt_status_offsets
is off-by-one. Fix this and also don't compared to a hard coded array
size but use adev->mode_info.num_hpd instead.
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c b/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c index ec5d8d93ad23..c8380f030400 100644 --- a/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c +++ b/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | |||
@@ -2782,7 +2782,7 @@ static int dce_v6_0_hpd_irq(struct amdgpu_device *adev, | |||
2782 | uint32_t disp_int, mask, int_control, tmp; | 2782 | uint32_t disp_int, mask, int_control, tmp; |
2783 | unsigned hpd; | 2783 | unsigned hpd; |
2784 | 2784 | ||
2785 | if (entry->src_data > 6) { | 2785 | if (entry->src_data >= adev->mode_info.num_hpd) { |
2786 | DRM_DEBUG("Unhandled interrupt: %d %d\n", entry->src_id, entry->src_data); | 2786 | DRM_DEBUG("Unhandled interrupt: %d %d\n", entry->src_id, entry->src_data); |
2787 | return 0; | 2787 | return 0; |
2788 | } | 2788 | } |