From 8773ad023eb28950eb0f802d2ca31a67f84adddc Mon Sep 17 00:00:00 2001 From: Keith Packard Date: Sat, 14 Jul 2007 08:47:50 -0700 Subject: [PATCH] Screen size bounds check in ProcRRSetCrtcConfig not masking out reflections. When checking how to validate the selected mode and position against the current screen size, the test against 90/270 rotation did not mask out reflection, so that when reflection was specified, the 90/270 test would never succeed. This caused incorrect bounds checking and would return an error to the user instead of rotating the screen. --- randr/rrcrtc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/randr/rrcrtc.c b/randr/rrcrtc.c index c8c2be264..3ce9e2155 100644 --- a/randr/rrcrtc.c +++ b/randr/rrcrtc.c @@ -794,7 +794,7 @@ ProcRRSetCrtcConfig (ClientPtr client) int source_width = mode->mode.width; int source_height = mode->mode.height; - if (rotation == RR_Rotate_90 || rotation == RR_Rotate_270) + if ((rotation & 0xf) == RR_Rotate_90 || (rotation & 0xf) == RR_Rotate_270) { source_width = mode->mode.height; source_height = mode->mode.width;