mlxfw: fix null-ptr-deref in mlxfw_mfa2_tlv_next()
authorNikita Zhandarovich <n.zhandarovich@fintech.ru>
Mon, 17 Apr 2023 12:07:18 +0000 (05:07 -0700)
committerPaolo Abeni <pabeni@redhat.com>
Tue, 18 Apr 2023 11:26:44 +0000 (13:26 +0200)
commitc0e73276f0fcbbd3d4736ba975d7dc7a48791b0c
treec5dedfc8b7d71123d91f9736200ef9ed0406e708
parent28e63d01a9f46d47b73045817f95c114a7769c71
mlxfw: fix null-ptr-deref in mlxfw_mfa2_tlv_next()

Function mlxfw_mfa2_tlv_multi_get() returns NULL if 'tlv' in
question does not pass checks in mlxfw_mfa2_tlv_payload_get(). This
behaviour may lead to NULL pointer dereference in 'multi->total_len'.
Fix this issue by testing mlxfw_mfa2_tlv_multi_get()'s return value
against NULL.

Found by Linux Verification Center (linuxtesting.org) with static
analysis tool SVACE.

Fixes: 410ed13cae39 ("Add the mlxfw module for Mellanox firmware flash process")
Co-developed-by: Natalia Petrova <n.petrova@fintech.ru>
Signed-off-by: Nikita Zhandarovich <n.zhandarovich@fintech.ru>
Reviewed-by: Ido Schimmel <idosch@nvidia.com>
Link: https://lore.kernel.org/r/20230417120718.52325-1-n.zhandarovich@fintech.ru
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
drivers/net/ethernet/mellanox/mlxfw/mlxfw_mfa2_tlv_multi.c