[PATCH] randr: Silence valgrind after server regen with panning enabled

Chris Wilson chris at chris-wilson.co.uk
Fri Mar 16 04:39:32 PDT 2012


If when bug hunting you trigger a server regeneration after enabling a
pnning mode without moving the nice, you cause a flurry of valgrind
warnings:

==28455== Conditional jump or move depends on uninitialised value(s)
==28455==    at 0x4D9431: xf86RandR13Pan (xf86RandR12.c:360)
==28455==    by 0x4DA4B1: xf86RandR12ScreenSetSize (xf86RandR12.c:725)
==28455==    by 0x4DA956: xf86RandR12CreateScreenResources
(xf86RandR12.c:841)
==28455==    by 0x4CB92B: xf86CrtcCreateScreenResources (xf86Crtc.c:728)
==28455==    by 0x425498: main (main.c:216)
==28455==  Uninitialised value was created by a heap allocation
==28455==    at 0x402894D: malloc (in
/usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==28455==    by 0x4DAA74: xf86RandR12Init (xf86RandR12.c:889)
==28455==    by 0x4CBADD: xf86CrtcScreenInit (xf86Crtc.c:782)
==28455==    by 0xA71DBAE: I830ScreenInit (intel_driver.c:1034)
==28455==    by 0x43CE0E: AddScreen (dispatch.c:3909)
==28455==    by 0x48FDE6: InitOutput (xf86Init.c:817)
==28455==    by 0x425400: main (main.c:204)

Since these depend upon the user exiting without moving the mouse in an
unsual setup, the impact of this bug is going to be very small.

Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
---
 hw/xfree86/modes/xf86RandR12.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/hw/xfree86/modes/xf86RandR12.c b/hw/xfree86/modes/xf86RandR12.c
index d5031a2..b383197 100644
--- a/hw/xfree86/modes/xf86RandR12.c
+++ b/hw/xfree86/modes/xf86RandR12.c
@@ -909,6 +909,7 @@ xf86RandR12Init (ScreenPtr pScreen)
     randrp->supported_rotations = RR_Rotate_0;
 
     randrp->maxX = randrp->maxY = 0;
+    randrp->pointerX = randrp->pointerY = 0; /* keep valgrind quiet */
 
     dixSetPrivate(&pScreen->devPrivates, xf86RandR12Key, randrp);
 
-- 
1.7.9.1



More information about the xorg-devel mailing list