enic: Validate length of nl attributes in enic_set_vf_port
authorRoded Zats <rzats@paloaltonetworks.com>
Wed, 22 May 2024 07:30:44 +0000 (10:30 +0300)
committerPaolo Abeni <pabeni@redhat.com>
Mon, 27 May 2024 09:18:01 +0000 (11:18 +0200)
commite8021b94b0412c37bcc79027c2e382086b6ce449
tree5e2720872cdc9e6558dae1623298f8fa339fe1da
parent0b4f5add9fa59bfd42c1030f572db2e4c395181b
enic: Validate length of nl attributes in enic_set_vf_port

enic_set_vf_port assumes that the nl attribute IFLA_PORT_PROFILE
is of length PORT_PROFILE_MAX and that the nl attributes
IFLA_PORT_INSTANCE_UUID, IFLA_PORT_HOST_UUID are of length PORT_UUID_MAX.
These attributes are validated (in the function do_setlink in rtnetlink.c)
using the nla_policy ifla_port_policy. The policy defines IFLA_PORT_PROFILE
as NLA_STRING, IFLA_PORT_INSTANCE_UUID as NLA_BINARY and
IFLA_PORT_HOST_UUID as NLA_STRING. That means that the length validation
using the policy is for the max size of the attributes and not on exact
size so the length of these attributes might be less than the sizes that
enic_set_vf_port expects. This might cause an out of bands
read access in the memcpys of the data of these
attributes in enic_set_vf_port.

Fixes: f8bd909183ac ("net: Add ndo_{set|get}_vf_port support for enic dynamic vnics")
Signed-off-by: Roded Zats <rzats@paloaltonetworks.com>
Link: https://lore.kernel.org/r/20240522073044.33519-1-rzats@paloaltonetworks.com
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/cisco/enic/enic_main.c