netdev - Re: [PATCH 4.14 stable v2 0/5] net: ip6 defrag: backport fixes

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: <CAFTs51UN8WOOQcQsXVZ6EooHJb5pynx3vOeCLC9RXtREZ2BjbA@mail.gmail.com>
Date:   Thu, 25 Apr 2019 09:01:54 -0700
From:   Peter Oskolkov <posk@...k.io>
To:     Lars Persson <lars.persson@...s.com>
Cc:     Peter Oskolkov <posk@...gle.com>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        stable@...r.kernel.org, linux-netdev <netdev@...r.kernel.org>,
        David Miller <davem@...emloft.net>,
        Eric Dumazet <edumazet@...gle.com>,
        Sasha Levin <sashal@...nel.org>,
        Captain Wiggum <captwiggum@...il.com>,
        Lars Persson <lists@...h.nu>
Subject: Re: [PATCH 4.14 stable v2 0/5] net: ip6 defrag: backport fixes

On Thu, Apr 25, 2019 at 1:13 AM Lars Persson <lars.persson@...s.com> wrote:
>
> On 4/23/19 7:48 PM, Peter Oskolkov wrote:
> > Lars Persson <lists@...h.nu> reported that a label was unused in
> > the previous version of this patchset, so I'm sending a v2 that fixes it.
> >
> > Sorry for the mess/v2.
> >
> > v2 changes: handle overlapping fragments the way it is done upstream.
> >
> > This is a backport of a 5.1rc patchset:
> >    https://patchwork.ozlabs.org/cover/1029418/
> >
> > Which was backported into 4.19:
> >    https://patchwork.ozlabs.org/cover/1081619/
> >
> > I had to backport two additional patches into 4.14 to make it work.
> >
> >
> > John Masinter (captwiggum), could you, please, confirm that this
> > patchset fixes TAHI tests? (I'm reasonably certain that it does, as
> > I ran ip_defrag selftest, but given the amount of changes here,
> > another set of completed tests would be nice to have).
> >
> >
> > Eric Dumazet (1):
> >    ipv6: frags: fix a lockdep false positive
> >
> > Florian Westphal (1):
> >    ipv6: remove dependency of nf_defrag_ipv6 on ipv6 module
> >
> > Peter Oskolkov (3):
> >    net: IP defrag: encapsulate rbtree defrag code into callable functions
> >    net: IP6 defrag: use rbtrees for IPv6 defrag
> >    net: IP6 defrag: use rbtrees in nf_conntrack_reasm.c
> >
> >   include/net/inet_frag.h                   |  16 +-
> >   include/net/ipv6.h                        |  29 --
> >   include/net/ipv6_frag.h                   | 111 +++++++
> >   net/ieee802154/6lowpan/reassembly.c       |   2 +-
> >   net/ipv4/inet_fragment.c                  | 293 +++++++++++++++++
> >   net/ipv4/ip_fragment.c                    | 302 +++---------------
> >   net/ipv6/netfilter/nf_conntrack_reasm.c   | 279 +++++------------
> >   net/ipv6/netfilter/nf_defrag_ipv6_hooks.c |   3 +-
> >   net/ipv6/reassembly.c                     | 364 ++++++----------------
> >   net/openvswitch/conntrack.c               |   1 +
> >   10 files changed, 627 insertions(+), 773 deletions(-)
> >   create mode 100644 include/net/ipv6_frag.h
> >
>
> Hi
>
> Our QA ran this with the IOL INTACT test suite and they give thumbs up.
>
> Tested-by: Lars Persson <lars.persson@...s.com>
>
> - Lars

Thanks, Lars! I'll prepare a patchset for 4.4 then - should be
identical, or very similar, to the 4.14 one.