netdev - Re: [PATCH bpf-next v3 00/13] bpf tc tunneling

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: <CAF=yD-K8+TZ7+BSFrj_cburNyfAdBXLMJOxDp0E6yHygYCr=bw@mail.gmail.com>
Date:   Sat, 23 Mar 2019 12:03:25 -0400
From:   Willem de Bruijn <willemdebruijn.kernel@...il.com>
To:     Daniel Borkmann <daniel@...earbox.net>
Cc:     Network Development <netdev@...r.kernel.org>,
        Alexei Starovoitov <ast@...nel.org>,
        Stanislav Fomichev <sdf@...gle.com>,
        Peter Oskolkov <posk@...gle.com>,
        Willem de Bruijn <willemb@...gle.com>
Subject: Re: [PATCH bpf-next v3 00/13] bpf tc tunneling

On Sat, Mar 23, 2019 at 6:53 AM Daniel Borkmann <daniel@...earbox.net> wrote:
>
> Hey Willem,
>
> On 03/22/2019 07:32 PM, Willem de Bruijn wrote:
> > From: Willem de Bruijn <willemb@...gle.com>
> >
> > BPF allows for dynamic tunneling, choosing the tunnel destination and
> > features on-demand. Extend bpf_skb_adjust_room to allow for efficient
> > tunneling at the TC hooks.
>
> Patch 9 in this series caused the following warning, please take a look
> for sending a follow-up fix, thanks.
>
> Report:
>
> tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/bpf/bpf-next.git  master
> branch HEAD: 7df5e3db8f6354125540dfa50affd2c02b7d2832  selftests: bpf: tc-bpf flow shaping with EDT
>
> Regressions in current branch:
>
> net/core/filter.c:3026:3: warning: 'mac_len' may be used uninitialized in this function [-Wmaybe-uninitialized]
> net/core/filter.c:3027:3: warning: 'mac_len' may be used uninitialized in this function [-Wmaybe-uninitialized]

That's surprising, sorry about that. Sent
http://patchwork.ozlabs.org/patch/1062325/ .