diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-01-13 02:49:00 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-02-01 02:33:03 -0500 |
commit | 5270c017f19f32812d7d83ba6e45873b8db503d0 (patch) | |
tree | e5b015ae3c3495695808ca27bccf4bbebf257cd8 /drivers/gpu/drm | |
parent | cfba2a001d0e36905016bb4f87fc47245c944c36 (diff) |
drm/vc4: fix a bounds check
commit 21ccc32496b2f63228f5232b3ac0e426e8fb3c31 upstream.
We accidentally return success even if vc4_full_res_bounds_check() fails.
Fixes: d5b1a78a772f ("drm/vc4: Add support for drawing 3D frames.")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Eric Engestrom <eric@engestrom.ch>
Reviewed-by: Eric Anholt <eric@anholt.net>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/gpu/drm')
-rw-r--r-- | drivers/gpu/drm/vc4/vc4_render_cl.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/vc4/vc4_render_cl.c b/drivers/gpu/drm/vc4/vc4_render_cl.c index 08886a309757..5cdd003605f5 100644 --- a/drivers/gpu/drm/vc4/vc4_render_cl.c +++ b/drivers/gpu/drm/vc4/vc4_render_cl.c | |||
@@ -461,7 +461,7 @@ static int vc4_rcl_surface_setup(struct vc4_exec_info *exec, | |||
461 | } | 461 | } |
462 | 462 | ||
463 | ret = vc4_full_res_bounds_check(exec, *obj, surf); | 463 | ret = vc4_full_res_bounds_check(exec, *obj, surf); |
464 | if (!ret) | 464 | if (ret) |
465 | return ret; | 465 | return ret; |
466 | 466 | ||
467 | return 0; | 467 | return 0; |