]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commit
net: bridge: fix vlan stats use-after-free on destruction
authorNikolay Aleksandrov <nikolay@cumulusnetworks.com>
Fri, 16 Nov 2018 16:50:01 +0000 (18:50 +0200)
committerDavid S. Miller <davem@davemloft.net>
Sun, 18 Nov 2018 05:38:44 +0000 (21:38 -0800)
commit9d332e69c1dc74dcd748de7cbd2dac5c61bda265
tree912c15036f8f473ddbf7526c4c9b4763c18f164f
parent95506588d2c1d72ca29adef8ae9bf771bcfb4ced
net: bridge: fix vlan stats use-after-free on destruction

Syzbot reported a use-after-free of the global vlan context on port vlan
destruction. When I added per-port vlan stats I missed the fact that the
global vlan context can be freed before the per-port vlan rcu callback.
There're a few different ways to deal with this, I've chosen to add a
new private flag that is set only when per-port stats are allocated so
we can directly check it on destruction without dereferencing the global
context at all. The new field in net_bridge_vlan uses a hole.

v2: cosmetic change, move the check to br_process_vlan_info where the
    other checks are done
v3: add change log in the patch, add private (in-kernel only) flags in a
    hole in net_bridge_vlan struct and use that instead of mixing
    user-space flags with private flags

Fixes: 9163a0fc1f0c ("net: bridge: add support for per-port vlan stats")
Reported-by: syzbot+04681da557a0e49a52e5@syzkaller.appspotmail.com
Signed-off-by: Nikolay Aleksandrov <nikolay@cumulusnetworks.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/bridge/br_private.h
net/bridge/br_vlan.c