From: Peter Spreadborough <peter.spreadboro...@broadcom.com>

This change addresses the CID 449059:  Integer handling issues
(INTEGER_OVERFLOW) reported by coverity.

Coverity issue: 449059
Signed-off-by: Peter Spreadborough <peter.spreadboro...@broadcom.com>
Reviewed-by: Ajit Khaparde <ajit.khapa...@broadcom.com>
---
 drivers/net/bnxt/hcapi/cfa_v3/mm/cfa_mm.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/net/bnxt/hcapi/cfa_v3/mm/cfa_mm.c 
b/drivers/net/bnxt/hcapi/cfa_v3/mm/cfa_mm.c
index 61fafadb20..05528dd3e4 100644
--- a/drivers/net/bnxt/hcapi/cfa_v3/mm/cfa_mm.c
+++ b/drivers/net/bnxt/hcapi/cfa_v3/mm/cfa_mm.c
@@ -123,7 +123,11 @@ int cfa_mm_open(void *cmm, struct cfa_mm_open_parms *parms)
        }
 
        for (i = 0; i < num_blocks; i++) {
-               context->blk_tbl[i].prev_blk_idx = i - 1;
+               if (i == 0)
+                       context->blk_tbl[i].prev_blk_idx = CFA_MM_INVALID32;
+               else
+                       context->blk_tbl[i].prev_blk_idx = i - 1;
+
                context->blk_tbl[i].next_blk_idx = i + 1;
                context->blk_tbl[i].num_free_records = records_per_block;
                context->blk_tbl[i].first_free_record = 0;
-- 
2.39.5 (Apple Git-154)

Reply via email to