linux-kernel - Re: [PATCH v3 0/3] uio: fix potential crash bug

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]

Date:   Fri, 6 Jul 2018 03:31:27 +0000
From:   Hamish Martin <Hamish.Martin@...iedtelesis.co.nz>
To:     "xiubli@...hat.com" <xiubli@...hat.com>,
        "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
CC:     "jannh@...gle.com" <jannh@...gle.com>,
        "pkalever@...hat.com" <pkalever@...hat.com>,
        "pkarampu@...hat.com" <pkarampu@...hat.com>,
        "atumball@...hat.com" <atumball@...hat.com>,
        "sabose@...hat.com" <sabose@...hat.com>,
        "mchristi@...hat.com" <mchristi@...hat.com>
Subject: Re: [PATCH v3 0/3] uio: fix potential crash bug

Hi Xiubo,

Thanks for your patch that addresses the regression found with my 
earlier commit.

I will take your code and run it in our scenario that showed the bug 
that led to my commit. Unfortunately I won't be able to get that done 
until mid-next week. I intend to report back to you by July 13th.

Thanks,
Hamish M

On 07/06/2018 02:57 PM, xiubli@...hat.com wrote:
> From: Xiubo Li <xiubli@...hat.com>
>
> The V2 patch set maybe not post successfully, so post it again
> with one new updation.
>
> V2:
> - resend it with some small fix
>
> V3:
> - switch to use request_threaded_irq
>
>
> Xiubo Li (3):
>    uio: use request_threaded_irq instead
>    uio: change to use the mutex lock instead of the spin lock
>    uio: fix crash after the device is unregistered
>
>   drivers/uio/uio.c          | 151 ++++++++++++++++++++++++++++++++++-----------
>   include/linux/uio_driver.h |   2 +-
>   2 files changed, 115 insertions(+), 38 deletions(-)
>