Commit c8dafb0e authored by Kalesh AP's avatar Kalesh AP Committed by Jakub Kicinski
Browse files

bnxt_en: Fix possible memory leak when hwrm_req_replace fails 



When hwrm_req_replace() fails, the driver is not invoking bnxt_req_drop()
which could cause a memory leak.

Fixes: bbf33d1d ("bnxt_en: update all firmware calls to use the new APIs")
Reviewed-by: default avatarPavan Chebbi <pavan.chebbi@broadcom.com>
Signed-off-by: default avatarKalesh AP <kalesh-anakkur.purayil@broadcom.com>
Signed-off-by: default avatarMichael Chan <michael.chan@broadcom.com>
Link: https://patch.msgid.link/20250104043849.3482067-2-michael.chan@broadcom.com


Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent 8c817eb2

drivers/net/ethernet/broadcom/bnxt/bnxt_ulp.c

+2 −1
Original line number Diff line number Diff line
@@ -208,7 +208,7 @@ int bnxt_send_msg(struct bnxt_en_dev *edev, 


	rc = hwrm_req_replace(bp, req, fw_msg->msg, fw_msg->msg_len);

	if (rc)

		return rc;

		goto drop_req;



	hwrm_req_timeout(bp, req, fw_msg->timeout);

	resp = hwrm_req_hold(bp, req);
@@ -220,6 +220,7 @@ int bnxt_send_msg(struct bnxt_en_dev *edev, 


		memcpy(fw_msg->resp, resp, resp_len);

	}

drop_req:

	hwrm_req_drop(bp, req);

	return rc;

}