[PATCH evdev] Fix absolute events with swapped axes

2011-12-14 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com We were correctly swapping the valuator values, but we were not calling valuator_mask_unset() when needed, so the cursor kept jumping to the edges. This patch does the swapping before the main for, so we don't need to store unswapped_{x,y} and

[PATCH evdev] Fix relative events with swapped axes

2011-12-15 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com After we swap the axes, we only call valuator_mask_set for axes that are not zero, so we need to unset the axes that became zero when swapped. Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- src/evdev.c |4 1 files changed, 4

[PATCH 1/3] glx: fix memory leak when destroying screen

2011-10-29 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 1,152 bytes in 1 blocks are definitely lost in loss record 536 of 575 at 0x4C25E84: calloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) by 0x483820: __glXScreenInit (glxscreens.c:357) by 0x48271C: __glXDRIscreenProbe

[PATCH 2/3] glx: don't leak fbconfigs

2011-10-29 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com 29,952 (208 direct, 29,744 indirect) bytes in 1 blocks are definitely lost in loss record 573 of 573 at 0x4C2779D: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) by 0x4829BC: createModeFromConfig (glxdricommon.c:131) by

[PATCH 1/5] dix: don't InitXTestDevices if there's no XTest extension

2011-11-01 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- dix/devices.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dix/devices.c b/dix/devices.c index 673a360..9d67c06 100644 --- a/dix/devices.c +++ b/dix/devices.c @@

[PATCH 2/5] Correctly free config file names

2011-11-01 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com We call xf86penConfigDirFiles twice, so we overwrite the configDirPath variable, losing the pointer. If we move the pointer management to the upper layer (the function callers), they will be able to call these functions as many times as they want, but

[PATCH 3/5] parser: free scandir's list

2011-11-01 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com It seems appropriate to make the function that frees the list elements also free the list. 80 bytes in 1 blocks are definitely lost in loss record 411 of 631 at 0x4C2779D: malloc (vgpreload_memcheck-amd64-linux.so) by 0x4C27927: realloc

[PATCH 4/5] parser: free val.str after xf86getBoolValue

2011-11-01 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com After we convert the value to a boolean, we discard the string. This is just one example: 3 bytes in 1 blocks are definitely lost in loss record 5 of 657 at 0x4C2779D: malloc (vgpreload_memcheck-amd64-linux.so) by 0x4D744D: xf86getToken

[PATCH 5/5] parser: free val.str after xstrtokenize

2011-11-01 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com After we tokenize val.str, we discard it. This is just one example: 6 bytes in 1 blocks are definitely lost in loss record 24 of 652 at 0x4C2779D: malloc (in vgpreload_memcheck-amd64-linux.so) by 0x4D744D: xf86getToken (scan.c:400) by

[PATCH 1/4] Correctly free config file names

2011-11-03 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com We call xf86penConfigDirFiles twice, so we overwrite the configDirPath variable, losing the pointer. If we move the pointer management to the upper layer (the function callers), they will be able to call these functions as many times as they want, but

[PATCH 2/4] parser: free scandir's list

2011-11-03 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com v2: move the free()s to the function that calls scandir 80 bytes in 1 blocks are definitely lost in loss record 411 of 631 at 0x4C2779D: malloc (vgpreload_memcheck-amd64-linux.so) by 0x4C27927: realloc (vgpreload_memcheck-amd64-linux.so) by

[PATCH] Fix segfault when there's no config dir

2011-11-25 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- hw/xfree86/parser/scan.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) Sorry. I guess this should be pushed soon... diff --git a/hw/xfree86/parser/scan.c

[PATCH] Fix segfault when there's no config dir

2011-11-28 Thread przanoni
From: Paulo Zanoni paulo.r.zan...@intel.com Also, call AddConfigDirFiles only if we found a file, protecting ourselves from future changes to the AddConfigDirFiles function. Signed-off-by: Paulo Zanoni paulo.r.zan...@intel.com --- hw/xfree86/parser/scan.c |9 ++--- 1 files changed, 6