linux-kernel - Re: [3.15-rc3] BUG: null ptr dereference in ichx_gpio_request

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <536C010F.3060504@hurleysoftware.com>
Date:	Thu, 08 May 2014 18:11:27 -0400
From:	Peter Hurley <peter@...leysoftware.com>
To:	Linus Walleij <linus.walleij@...aro.org>,
	Lee Jones <lee.jones@...aro.org>
CC:	Vincent Donnefort <vdonnefort@...il.com>,
	Linux kernel <linux-kernel@...r.kernel.org>
Subject: Re: [3.15-rc3] BUG: null ptr dereference in ichx_gpio_request_regions()

On 05/08/2014 05:48 PM, Linus Walleij wrote:
> On Wed, May 7, 2014 at 4:33 PM, Peter Hurley <peter@...leysoftware.com> 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@...il.com>
>>>> 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@...aro.org>
>>>>       Signed-off-by: Vincent Donnefort <vdonnefort@...il.com>
>>>>       Signed-off-by: Lee Jones <lee.jones@...aro.org>
>>>>
>>>
>>> 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@...il.com> wrote:
 >
 >> From: Vincent Donnefort <vdonnefort@...il.com>
 >>
 >> 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@...il.com>
 >
 > 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/