Re: [PATCHv2 1/1] omap: Ptr "isr_reg" tracked as NULL wasdereferenced

From: Tony Lindgren
Date: Mon Oct 18 2010 - 19:12:10 EST


* Evgeny Kuznetsov <EXT-Eugeny.Kuznetsov@xxxxxxxxx> [101014 23:11]:
> 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:
> Hi Tony,
>
> It is not a bug fix, it is just check to prevent potential issues. Used
> to warn in case of bug and prevent kernel oops. Check added only here
> (not all gpio code cleanup) since here it could cause kernel opps.
>
> Would you apply patch if I will leave only if condition section in
> patch?
>
> if (WARN_ON(!isr_reg))
> goto exit;
>
> If yes, I'll resend v3 patch.

Sounds like a valid check to me.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/