| commit | 041bf0225552044b85ce7ee7981e790d987d6ceb | |
| author | Dan Carpenter <dan.carpenter@oracle.com> | |
| Thu, 16 Jun 2016 08:30:23 +0000 (16 11:30 +0300) | ||
| committer | Alex Deucher <alexander.deucher@amd.com> | |
| Fri, 17 Jun 2016 17:50:10 +0000 (17 13:50 -0400) | ||
| tree | 178d8d94fb6f673e9681eae1429fc0e5c65780ad | treesnapshot (tar.gz zip) |
| parent | 0ab15bdeb2943bd6491a35ec4eeb53a9a4436525 | commitdiff |
drm/amdgpu: missing bounds check in amdgpu_set_pp_force_state()
There is no limit on high "idx" can go. It should be less than
ARRAY_SIZE(data.states) which is 16.
The "data" variable wasn't declared in that scope so I shifted the code
around a bit to make it work. Also I made "idx" unsigned.
Fixes: f3898ea12fc1 ('drm/amd/powerplay: add some sysfs interfaces for powerplay.')
Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
There is no limit on high "idx" can go. It should be less than
ARRAY_SIZE(data.states) which is 16.
The "data" variable wasn't declared in that scope so I shifted the code
around a bit to make it work. Also I made "idx" unsigned.
Fixes: f3898ea12fc1 ('drm/amd/powerplay: add some sysfs interfaces for powerplay.')
Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
| drivers/gpu/drm/amd/amdgpu/amdgpu_pm.c | diffblobblamehistory |