Re: [PATCH] dix: check for NULL pointer before dereferences it in DeviceEnterLeaveEvent

2010-04-21 Thread Jeremy Huddleston
This patch also adds this change which isn't really described by the commit description: grab = mouse-deviceGrab.grab; but you forgot to change this: GrabPtr grab = mouse-deviceGrab.grab; to GrabPtr grab; On Apr 19, 2010, at 10:58, Tiago Vignatti wrote:

Re: [PATCH] dix: check for NULL pointer before dereferences it in DeviceEnterLeaveEvent

2010-04-19 Thread Peter Hutterer
On Mon, Apr 19, 2010 at 08:58:04PM +0300, Tiago Vignatti wrote: mouse is already used before its checking should be performed. So check on the beginning instead. --- dix/events.c |6 +- 1 files changed, 5 insertions(+), 1 deletions(-) diff --git a/dix/events.c b/dix/events.c

Re: [PATCH] dix: check for NULL pointer before dereferences it in DeviceEnterLeaveEvent

2010-04-18 Thread Peter Hutterer
On Fri, Apr 16, 2010 at 06:42:23PM +0300, Tiago Vignatti wrote: Both mouse and mouse-button are already used before their checking were being performed. So check on the beginning. Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com --- dix/events.c |8 ++-- 1 files changed, 6