]> git.dujemihanovic.xyz Git - linux.git/commit
rtnetlink: Correct nested IFLA_VF_VLAN_LIST attribute validation
authorRoded Zats <rzats@paloaltonetworks.com>
Thu, 2 May 2024 15:57:51 +0000 (18:57 +0300)
committerJakub Kicinski <kuba@kernel.org>
Fri, 3 May 2024 22:57:50 +0000 (15:57 -0700)
commit1aec77b2bb2ed1db0f5efc61c4c1ca3813307489
tree3743fd5e15bdd452ee72ff6a1725e25aed1c07a5
parentd0de61673908765f4e189fe26f4de6e87caf00e5
rtnetlink: Correct nested IFLA_VF_VLAN_LIST attribute validation

Each attribute inside a nested IFLA_VF_VLAN_LIST is assumed to be a
struct ifla_vf_vlan_info so the size of such attribute needs to be at least
of sizeof(struct ifla_vf_vlan_info) which is 14 bytes.
The current size validation in do_setvfinfo is against NLA_HDRLEN (4 bytes)
which is less than sizeof(struct ifla_vf_vlan_info) so this validation
is not enough and a too small attribute might be cast to a
struct ifla_vf_vlan_info, this might result in an out of bands
read access when accessing the saved (casted) entry in ivvl.

Fixes: 79aab093a0b5 ("net: Update API for VF vlan protocol 802.1ad support")
Signed-off-by: Roded Zats <rzats@paloaltonetworks.com>
Reviewed-by: Donald Hunter <donald.hunter@gmail.com>
Link: https://lore.kernel.org/r/20240502155751.75705-1-rzats@paloaltonetworks.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/core/rtnetlink.c