Re: [PATCH] drm/gud: validate GUD_ROTATION_0 is present in supported rotations

Ruben Wauters <[email protected]> Sun, 23 Aug 2026 17:33:35 +0100

Newsgroups	org.freedesktop.lists.dri-devel,org.kernel.vger.linux-kernel
Message-ID	<[email protected]>

On Fri, 2026-08-21 at 12:46 +0530, Sajal Gupta wrote:
> The rotation argument to drm_plane_create_rotation_property() is set to
> DRM_MODE_ROTATE_0, and the device reported rotation bitmask is used as
> the supported_rotations argument. The driver never validates that
> GUD_ROTATION_0 is present, so a device that omits it from its
> GUD_PROPERTY_ROTATION triggers the
> WARN_ON(rotation & ~supported_rotations) in
> drm_plane_create_rotation_property()
> 
> Fix this by skipping the creation of rotation property if the device
> doesn't have the GUD_ROTATION_0 bit

Hello, Thank you for the patch
> 
> Fixes: 40e1a70b4aed ("drm: Add GUD USB Display driver")
> Reported-by: [email protected]
> Closes: https://syzkaller.appspot.com/bug?extid=efe2810681f1b065d3a8
> Tested-by: [email protected]
> Signed-off-by: Sajal Gupta <[email protected]>
Acked-by: Ruben Wauters <[email protected]>
> ---
>  drivers/gpu/drm/gud/gud_drv.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/gpu/drm/gud/gud_drv.c b/drivers/gpu/drm/gud/gud_drv.c
> index 89bd6ca36003..3a1b9e2a2eaa 100644
> --- a/drivers/gpu/drm/gud/gud_drv.c
> +++ b/drivers/gpu/drm/gud/gud_drv.c
> @@ -289,6 +289,8 @@ static int gud_plane_add_properties(struct gud_device *gdrm)
>  			 * but mask out any additions on future devices.
>  			 */
>  			val &= GUD_ROTATION_MASK;
> +			if (!(val & GUD_ROTATION_0))
> +				continue;
>  			ret = drm_plane_create_rotation_property(&gdrm->plane,
>  								 DRM_MODE_ROTATE_0, val);
>  			break;
> --
> 2.55.0

signature.asc (application/pgp-signature, 870 B)