]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
geneve: require CAP_NET_ADMIN in the device netns for changelink
authorDoruk Tan Ozturk <doruk@0sec.ai>
Thu, 16 Jul 2026 20:35:00 +0000 (22:35 +0200)
committerJakub Kicinski <kuba@kernel.org>
Thu, 23 Jul 2026 15:17:09 +0000 (08:17 -0700)
commit8efb8f8bbb353b8f2fdf4f37534c6d96c9f69e01
treee845bb1f7a1a001128f822f73d8569df39d6a1eb
parent3a61bd9637f3d929aa846e4eb3d98b48c26fcb0e
geneve: require CAP_NET_ADMIN in the device netns for changelink

A tunnel changelink() operates on at most two netns, dev_net(dev) and
the sticky underlay netns geneve->net. They differ once the device is
created in or moved to a netns other than the one the request runs in.
The rtnl changelink path checks CAP_NET_ADMIN only against dev_net(dev),
so a caller privileged there but not in geneve->net can rewrite a geneve
device whose underlay lives in geneve->net.

geneve_changelink() applies the new configuration against geneve->net:
geneve_link_config() and the geneve_quiesce()/geneve_unquiesce() pair
reopen the underlay sockets in that netns (geneve_sock_add() uses
geneve->net), so the same reasoning as the tunnel changelink series
applies here.

Gate geneve_changelink() with rtnl_dev_link_net_capable(), at the top of
the op before any attribute is parsed, matching ipgre_changelink() and
the rest of the "require CAP_NET_ADMIN in the device netns for
changelink" series.

Found by 0sec automated security-research tooling (https://0sec.ai).

Fixes: 5b861f6baa3a ("geneve: add rtnl changelink support")
Cc: stable@vger.kernel.org
Signed-off-by: Doruk Tan Ozturk <doruk@0sec.ai>
Reviewed-by: Fernando Fernandez Mancera <fmancera@suse.de>
Link: https://patch.msgid.link/20260716203500.70573-3-doruk@0sec.ai
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
drivers/net/geneve.c