From: Michael Chan <michael.chan@broadcom.com>

Date: Sun, 26 Apr 2020 16:24:41 -0400

Subject: bnxt_en: Return error when allocating zero size context memory.

Patch-mainline: v5.7-rc5

Git-commit: bbf211b1ecb891c7e0cc7888834504183fc8b534

References: bsc#1171150

bnxt_alloc_ctx_pg_tbls() should return error when the memory size of the

context memory to set up is zero.  By returning success (0), the caller

may proceed normally and may crash later when it tries to set up the

memory.

Fixes: 08fe9d181606 ("bnxt_en: Add Level 2 context memory paging support.")

Signed-off-by: Michael Chan <michael.chan@broadcom.com>

Signed-off-by: David S. Miller <davem@davemloft.net>

Acked-by: Thomas Bogendoerfer <tbogendoerfer@suse.de>

---

 drivers/net/ethernet/broadcom/bnxt/bnxt.c |    2 +-

 1 file changed, 1 insertion(+), 1 deletion(-)

--- a/drivers/net/ethernet/broadcom/bnxt/bnxt.c

+++ b/drivers/net/ethernet/broadcom/bnxt/bnxt.c

@@ -6662,7 +6662,7 @@ static int bnxt_alloc_ctx_pg_tbls(struct

 	int rc;

 	if (!mem_size)

-		return 0;

+		return -EINVAL;

 	ctx_pg->nr_pages = DIV_ROUND_UP(mem_size, BNXT_PAGE_SIZE);

 	if (ctx_pg->nr_pages > MAX_CTX_TOTAL_PAGES) {