diff options
author | Leo Liu <leo.liu@amd.com> | 2014-04-28 09:40:22 -0400 |
---|---|---|
committer | Christian König <christian.koenig@amd.com> | 2014-05-01 06:28:06 -0400 |
commit | 695daf1a8e731a4b5b89de89a61f32a4d7ad7094 (patch) | |
tree | 53ae27387f131279f27af15c837aa4b6896e6e1a | |
parent | f5d636d2a74b755879feec35e14a259de52ccc07 (diff) |
drm/radeon: check buffer relocation offset
Signed-off-by: Leo Liu <leo.liu@amd.com>
Signed-off-by: Christian König <christian.koenig@amd.com>
Cc: stable@vger.kernel.org
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_uvd.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_uvd.c b/drivers/gpu/drm/radeon/radeon_uvd.c index 5748bdaeacce..0f96c471c6d8 100644 --- a/drivers/gpu/drm/radeon/radeon_uvd.c +++ b/drivers/gpu/drm/radeon/radeon_uvd.c | |||
@@ -465,6 +465,10 @@ static int radeon_uvd_cs_reloc(struct radeon_cs_parser *p, | |||
465 | cmd = radeon_get_ib_value(p, p->idx) >> 1; | 465 | cmd = radeon_get_ib_value(p, p->idx) >> 1; |
466 | 466 | ||
467 | if (cmd < 0x4) { | 467 | if (cmd < 0x4) { |
468 | if (end <= start) { | ||
469 | DRM_ERROR("invalid reloc offset %X!\n", offset); | ||
470 | return -EINVAL; | ||
471 | } | ||
468 | if ((end - start) < buf_sizes[cmd]) { | 472 | if ((end - start) < buf_sizes[cmd]) { |
469 | DRM_ERROR("buffer (%d) to small (%d / %d)!\n", cmd, | 473 | DRM_ERROR("buffer (%d) to small (%d / %d)!\n", cmd, |
470 | (unsigned)(end - start), buf_sizes[cmd]); | 474 | (unsigned)(end - start), buf_sizes[cmd]); |