linux-kernel - Re: [PATCH] net/dccp: fix use after free in tw_timer

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: <ce0ad8f1-f0df-0853-92ce-7dbb22c94698@virtuozzo.com>
Date:   Wed, 22 Feb 2017 12:35:20 +0300
From:   Andrey Ryabinin <aryabinin@...tuozzo.com>
To:     David Miller <davem@...emloft.net>
CC:     <gerrit@....abdn.ac.uk>, <dccp@...r.kernel.org>,
        <dvyukov@...gle.com>, <edumazet@...gle.com>,
        <xiyou.wangcong@...il.com>, <kuznet@....inr.ac.ru>,
        <yoshfuji@...ux-ipv6.org>, <kaber@...sh.net>,
        <syzkaller@...glegroups.com>, <netdev@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] net/dccp: fix use after free in tw_timer_handler()

On 02/21/2017 09:24 PM, David Miller wrote:
> From: David Miller <davem@...emloft.net>
> Date: Tue, 21 Feb 2017 13:23:51 -0500 (EST)
> 
>> From: Andrey Ryabinin <aryabinin@...tuozzo.com>
>> Date: Tue, 21 Feb 2017 14:27:40 +0300
>>
>>> DCCP doesn't purge timewait sockets on network namespace shutdown.
>>> So, after net namespace destroyed we could still have an active timer
>>> which will trigger use after free in tw_timer_handler():
>>  ...
>>> Add .exit_batch hook to dccp_v4_ops()/dccp_v6_ops() which will purge
>>> timewait sockets on net namespace destruction and prevent above issue.
>>>
>>> Reported-by: Dmitry Vyukov <dvyukov@...gle.com>
>>> Signed-off-by: Andrey Ryabinin <aryabinin@...tuozzo.com>
>>
>> Applied and queued up for -stable, thanks.
> 
> Actually, this doesn't even compile.  Please fix this up and resubmit:
> 

Right, I tested this on top of the Linus' tree. Rebased on -next now.