[PATCH] xfree86: check for NULL pointer before dereferences it in parser code

2010-04-19 Thread Tiago Vignatti
Seems to be harmless. Meh. Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com --- hw/xfree86/parser/scan.c |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/hw/xfree86/parser/scan.c b/hw/xfree86/parser/scan.c index 8aab0cf..06710d5 100644 ---

Re: [PATCH] xfree86: check for NULL pointer before dereferences it in parser code

2010-04-19 Thread Peter Hutterer
On Mon, Apr 19, 2010 at 08:58:02PM +0300, Tiago Vignatti wrote: Seems to be harmless. Meh. Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com --- hw/xfree86/parser/scan.c |7 +-- 1 files changed, 5 insertions(+), 2 deletions(-) diff --git a/hw/xfree86/parser/scan.c

Re: [PATCH] xfree86: check for NULL pointer before dereferences it in parser code

2010-04-19 Thread Dan Nicholson
On Mon, Apr 19, 2010 at 10:58 AM, Tiago Vignatti tiago.vigna...@nokia.com wrote: Seems to be harmless. Meh. Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com ---  hw/xfree86/parser/scan.c |    7 +--  1 files changed, 5 insertions(+), 2 deletions(-) diff --git

Re: [PATCH] xfree86: check for NULL pointer before dereferences it in parser code

2010-04-17 Thread Keith Packard
On Fri, 16 Apr 2010 18:42:19 +0300, Tiago Vignatti tiago.vigna...@nokia.com wrote: - if (!name || name[0] == '.' || len = suflen) + if (!name) + return 0; + + name = de-d_name; You might want to assign name before testing it. -- keith.pack...@intel.com

Re: [PATCH] xfree86: check for NULL pointer before dereferences it in parser code

2010-04-16 Thread Alan Coopersmith
Tiago Vignatti wrote: Seems to be harmless. Meh. Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com --- hw/xfree86/parser/scan.c |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/hw/xfree86/parser/scan.c b/hw/xfree86/parser/scan.c index

Re: [PATCH] xfree86: check for NULL pointer before dereferences it in parser code

2010-04-16 Thread Éric Piel
On 16/04/10 18:17, Alan Coopersmith wrote: Tiago Vignatti wrote: Seems to be harmless. Meh. Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com --- hw/xfree86/parser/scan.c |9 +++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/hw/xfree86/parser/scan.c

Re: [PATCH] xfree86: check for NULL pointer before dereferences it in parser code

2010-04-16 Thread Dan Nicholson
On Fri, Apr 16, 2010 at 10:16 AM, Éric Piel e.a.b.p...@tudelft.nl wrote: On 16/04/10 18:17, Alan Coopersmith wrote: Tiago Vignatti wrote: Seems to be harmless. Meh. Signed-off-by: Tiago Vignatti tiago.vigna...@nokia.com ---  hw/xfree86/parser/scan.c |    9 +++--  1 files changed, 7