From: Pietro Borrello Date: Sat, 28 Jan 2023 16:29:17 +0000 (+0000) Subject: net/tls: tls_is_tx_ready() checked list_entry X-Git-Tag: baikal/aarch64/sdk6.1~146 X-Git-Url: https://git.baikalelectronics.ru/sdk/?a=commitdiff_plain;h=5dd2ab2e02df14b3919d2f683f3f07b0c15ea00f;p=kernel.git net/tls: tls_is_tx_ready() checked list_entry [ Upstream commit ffe2a22562444720b05bdfeb999c03e810d84cbb ] tls_is_tx_ready() checks that list_first_entry() does not return NULL. This condition can never happen. For empty lists, list_first_entry() returns the list_entry() of the head, which is a type confusion. Use list_first_entry_or_null() which returns NULL in case of empty lists. Fixes: 4e09428b5ad7 ("net/tls: Add support for async encryption of records for performance") Signed-off-by: Pietro Borrello Link: https://lore.kernel.org/r/20230128-list-entry-null-check-tls-v1-1-525bbfe6f0d0@diag.uniroma1.it Signed-off-by: Jakub Kicinski Signed-off-by: Sasha Levin --- diff --git a/net/tls/tls_sw.c b/net/tls/tls_sw.c index 9ed9786341259..a83d2b4275fa6 100644 --- a/net/tls/tls_sw.c +++ b/net/tls/tls_sw.c @@ -2427,7 +2427,7 @@ static bool tls_is_tx_ready(struct tls_sw_context_tx *ctx) { struct tls_rec *rec; - rec = list_first_entry(&ctx->tx_list, struct tls_rec, list); + rec = list_first_entry_or_null(&ctx->tx_list, struct tls_rec, list); if (!rec) return false;