netdev - Re: [PATCH net v2] ipv4: Fix use-after-free when flushing FIB tables

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] [day] [month] [year] [list]

Message-ID: <20171220174117.GA11654@splinter>
Date:   Wed, 20 Dec 2017 19:41:17 +0200
From:   Ido Schimmel <idosch@...sch.org>
To:     Alexander Duyck <alexander.duyck@...il.com>
Cc:     Ido Schimmel <idosch@...lanox.com>,
        Netdev <netdev@...r.kernel.org>,
        David Miller <davem@...emloft.net>,
        "Duyck, Alexander H" <alexander.h.duyck@...el.com>,
        David Ahern <dsahern@...il.com>,
        Fengguang Wu <fengguang.wu@...el.com>, mlxsw@...lanox.com
Subject: Re: [PATCH net v2] ipv4: Fix use-after-free when flushing FIB tables

On Wed, Dec 20, 2017 at 07:32:21AM -0800, Alexander Duyck wrote:
> I think we might want even more description here. Specifically why
> reversing the order allows local to be destroyed before main. I was
> thinking something along the lines of:
> 
> Destroy the tables in reverse order to guarantee that the local table,
> ID 255, is destroyed before main table, ID 254. This is necessary as
> local may contain references to data contained in main.

Sent v3:
http://patchwork.ozlabs.org/patch/851564/

Thanks