[PATCHv3 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-11-08 Thread Evgeny Kuznetsov
From: Evgeny Kuznetsov ext-eugeny.kuznet...@nokia.com Value of isr_reg pointer is depend on configuration and GPIO method. Potentially it may have NULL value and it is dereferenced later in code. Warning and exit from function are added in this case. Signed-off-by: Evgeny Kuznetsov ext

RE: [PATCHv2 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-15 Thread Evgeny Kuznetsov
On Wed, 2010-10-13 at 16:09 +0400, Evgeny Kuznetsov wrote: On Wed, 2010-10-13 at 17:20 +0530, ext Varadarajan, Charulatha wrote: snip From: Evgeny Kuznetsov ext-eugeny.kuznet...@nokia.com Value of isr_reg pointer is depend on configuration and GPIO method

[PATCHv2 0/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-13 Thread Evgeny Kuznetsov
. And added WARN_ON() macro and exit from function if pointer isr_reg is NULL. Also compilation check is added for correct architecture configuration. Thanks, Evgeny Evgeny Kuznetsov (1): omap: Ptr isr_reg tracked as NULL was dereferenced arch/arm/plat-omap/gpio.c | 18 ++ 1 files

[PATCHv2 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-13 Thread Evgeny Kuznetsov
From: Evgeny Kuznetsov ext-eugeny.kuznet...@nokia.com Value of isr_reg pointer is depend on configuration and GPIO method. Potentially it may have NULL value and it is dereferenced later in code. If pointer is NULL there is some kernel issue. Warning and exit from function are added in this case

RE: [PATCHv2 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-13 Thread Evgeny Kuznetsov
On Wed, 2010-10-13 at 16:25 +0530, ext Varadarajan, Charulatha wrote: -Original Message- From: Evgeny Kuznetsov [mailto:ext-eugeny.kuznet...@nokia.com] Sent: Wednesday, October 13, 2010 3:35 PM To: t...@atomide.com Cc: linux-omap@vger.kernel.org; linux-ker...@vger.kernel.org

RE: [PATCHv2 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-13 Thread Evgeny Kuznetsov
On Wed, 2010-10-13 at 17:20 +0530, ext Varadarajan, Charulatha wrote: snip From: Evgeny Kuznetsov ext-eugeny.kuznet...@nokia.com Value of isr_reg pointer is depend on configuration and GPIO method. Potentially it may have NULL value and it is dereferenced later in code

Re: [PATCH 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-08 Thread Evgeny Kuznetsov
On Wed, 2010-10-06 at 10:33 +0400, Evgeny Kuznetsov wrote: On Tue, 2010-10-05 at 08:01 -0700, ext Kevin Hilman wrote: Felipe Balbi ba...@ti.com writes: Hi, On Tue, Oct 05, 2010 at 03:42:10AM -0500, Evgeny Kuznetsov wrote: + if (!isr_reg) { + printk(KERN_ERR FATAL

Re: [PATCH 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-06 Thread Evgeny Kuznetsov
On Tue, 2010-10-05 at 08:01 -0700, ext Kevin Hilman wrote: Felipe Balbi ba...@ti.com writes: Hi, On Tue, Oct 05, 2010 at 03:42:10AM -0500, Evgeny Kuznetsov wrote: + if (!isr_reg) { + printk(KERN_ERR FATAL: Incorrect GPIO method %i\n, + bank-method

[PATCH 0/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-05 Thread Evgeny Kuznetsov
Hi, Here is patch which fixes bug in /arch/arm/plat-omap/gpio.c file. Pointer which may have NULL value in some cases (depend on kernel configuration and GPIO method) is dereferenced later in code. If pointer is NULL it mean some kernel bug. Thanks, Best Regards, Evgeny Evgeny Kuznetsov (1

[PATCH 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-05 Thread Evgeny Kuznetsov
From: Evgeny Kuznetsov ext-eugeny.kuznet...@nokia.com Value of isr_reg pointer is depend on configuration and GPIO method. Potentially it may have NULL value and it is dereferenced later in code. If pointer is NULL there is some kernel bug. Log line and 'BUG' macro are added to halt code

Re: [PATCH 1/1] omap: Ptr isr_reg tracked as NULL was dereferenced

2010-10-05 Thread Evgeny Kuznetsov
Hi, Yes, but no log in that case. Regards, Evgeny On Tue, 2010-10-05 at 12:32 +0300, ext Felipe Balbi wrote: Hi, On Tue, Oct 05, 2010 at 03:42:10AM -0500, Evgeny Kuznetsov wrote: +if (!isr_reg) { +printk(KERN_ERR FATAL: Incorrect GPIO method %i\n