]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
drm/rockchip: vop2: use correct destination rectangle height check
authorAlok Tiwari <alok.a.tiwari@oracle.com>
Sun, 12 Oct 2025 14:20:01 +0000 (07:20 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 29 Oct 2025 13:04:34 +0000 (14:04 +0100)
[ Upstream commit 7f38a1487555604bc4e210fa7cc9b1bce981c40e ]

The vop2_plane_atomic_check() function incorrectly checks
drm_rect_width(dest) twice instead of verifying both width and height.
Fix the second condition to use drm_rect_height(dest) so that invalid
destination rectangles with height < 4 are correctly rejected.

Fixes: 604be85547ce ("drm/rockchip: Add VOP2 driver")
Signed-off-by: Alok Tiwari <alok.a.tiwari@oracle.com>
Reviewed-by: Andy Yan <andy.yan@rock-chips.com>
Signed-off-by: Heiko Stuebner <heiko@sntech.de>
Link: https://lore.kernel.org/r/20251012142005.660727-1-alok.a.tiwari@oracle.com
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/gpu/drm/rockchip/rockchip_drm_vop2.c

index 6efa0a51b7d657a3d72df7a39ed053f651c73119..e14557d80efc2d7bb2288a3f44724ea85e38d639 100644 (file)
@@ -983,7 +983,7 @@ static int vop2_plane_atomic_check(struct drm_plane *plane,
                return format;
 
        if (drm_rect_width(src) >> 16 < 4 || drm_rect_height(src) >> 16 < 4 ||
-           drm_rect_width(dest) < 4 || drm_rect_width(dest) < 4) {
+           drm_rect_width(dest) < 4 || drm_rect_height(dest) < 4) {
                drm_err(vop2->drm, "Invalid size: %dx%d->%dx%d, min size is 4x4\n",
                        drm_rect_width(src) >> 16, drm_rect_height(src) >> 16,
                        drm_rect_width(dest), drm_rect_height(dest));