xserver: Branch 'master'

Keith Packard keithp at kemper.freedesktop.org
Mon Apr 21 22:39:34 PDT 2014


 hw/xfree86/modes/xf86RandR12.c |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 70e564104b69bc53d29633f392f2c1ab94caddc9
Author: Dominik Behr <dbehr at chromium.org>
Date:   Tue Apr 1 20:36:13 2014 -0700

    xf86RandR12: use correct gamma size when allocating gamma table
    
    When setting crtc->gamma_size to randr_crtc->gammaSize we should
    use randr_crtc->gammaSize to allocate new gamma table in crtc.
    Currently, if randr_crtc->gammaSize > crtc->gammaSize the subsequent
    memcpy will overwrite memory beyond the end of gamma table.
    
    Signed-off-by: Dominik Behr <dbehr at chromium.org>
    Reviewed-by: Stéphane Marchesin <marcheu at chromium.org>
    Signed-off-by: Keith Packard <keithp at keithp.com>

diff --git a/hw/xfree86/modes/xf86RandR12.c b/hw/xfree86/modes/xf86RandR12.c
index 66139dc..8a04dfc 100644
--- a/hw/xfree86/modes/xf86RandR12.c
+++ b/hw/xfree86/modes/xf86RandR12.c
@@ -1256,12 +1256,13 @@ xf86RandR12CrtcSetGamma(ScreenPtr pScreen, RRCrtcPtr randr_crtc)
         CARD16 *tmp_ptr;
 
         tmp_ptr =
-            realloc(crtc->gamma_red, 3 * crtc->gamma_size * sizeof(CARD16));
+            realloc(crtc->gamma_red,
+                    3 * randr_crtc->gammaSize * sizeof(CARD16));
         if (!tmp_ptr)
             return FALSE;
         crtc->gamma_red = tmp_ptr;
-        crtc->gamma_green = crtc->gamma_red + crtc->gamma_size;
-        crtc->gamma_blue = crtc->gamma_green + crtc->gamma_size;
+        crtc->gamma_green = crtc->gamma_red + randr_crtc->gammaSize;
+        crtc->gamma_blue = crtc->gamma_green + randr_crtc->gammaSize;
     }
 
     crtc->gamma_size = randr_crtc->gammaSize;


More information about the xorg-commit mailing list