Re: [3.15-rc3] BUG: null ptr dereference in ichx_gpio_request_regions()

[Peter Hurley]

On 05/08/2014 05:48 PM, Linus Walleij wrote:
> On Wed, May 7, 2014 at 4:33 PM, Peter Hurley <peter@xxxxxxxxxxxxxxxxxx> wrote:
> > Hi Vincent,
> >
> > On 05/07/2014 10:05 AM, Vincent Donnefort wrote:
> > > Hello Peter,
> >
> >
> > > > This is almost certainly caused by the uninitialized regs ptr
> > > > in the ich6_desc struct (i3100_desc struct has the same problem)
> > > > introduced in this commit:
> > > >
> > > > commit bb62a35bd5d96e506af0ea8dd145480b9172a2a6
> > > > Author: Vincent Donnefort <vdonnefort@xxxxxxxxx>
> > > > Date:   Fri Feb 14 15:01:56 2014 +0100
> > > >
> > > >       gpio: ich: Add support for multiple register addresses
> > > >
> > > >       This patch introduces regs and reglen pointers which allow a chipset
> > > > to have
> > > >       register addresses differing from ICH ones.
> > > >
> > > >       Acked-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> > > >       Signed-off-by: Vincent Donnefort <vdonnefort@xxxxxxxxx>
> > > >       Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx>
> > >
> > > Yes indeed, this must be linked to this thread
> > > https://lkml.org/lkml/2014/4/15/292
> >
> >
> > Thanks for the link.
> >
> > I searched through LKML but without the driver name or the offending
> > function
> > in the commit message I couldn't find it.
> >
> > Linus,
> >
> > What tree is this bug fix trapped in? I see it didn't make -rc4 either.
>
> Since the last signoff is Lee I guess it's in the MFD tree?

The commit above is the buggy commit. I don't have a commit id for
the bug fix since I don't know what tree it's stuck in. Looking over
Lee's tree, I don't see it.

Below is your signoff on LKML of the fix.

On 04/22/2014 05:15 PM, Linus Walleij wrote:
> On Tue, Apr 15, 2014 at 2:21 PM, Vincent Donnefort <vdonnefort@xxxxxxxxx> wrote:
>
>> From: Vincent Donnefort <vdonnefort@xxxxxxxxx>
>>
>> This patch fixes kernel NULL pointer BUG introduced by the following commit:
>> b667cf488aa9476b0ab64acd91f2a96f188cfd21
>> gpio: ich: Add support for multiple register addresses.
>>
>> Signed-off-by: Vincent Donnefort <vdonnefort@xxxxxxxxx>
>
> Patch applied for fixes with Eric's tested-by tag.

Please expedite this fix as it has already missed -rc3 & -rc4.

Regards,
Peter Hurley
--
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/