From d756ff1530dce2f08f25b9f132420160e71c31f2 Mon Sep 17 00:00:00 2001 From: sheenam monga Date: Mon, 15 Apr 2019 15:43:18 +0530 Subject: [PATCH] qcacld-3.0: Fix null ptr dereference in lim_process_assoc_cleanup Currently, lim_process_assoc_cleanup does not do null validation for sta_ds which can lead to null pointer derefrence. Add null pointer check for sta_ds in lim_process_assoc_cleanup Change-Id: I74ad0e11213c0bec1473984f312f18b5f1082082 CRs-Fixed: 2423883 --- core/mac/src/pe/lim/lim_process_assoc_req_frame.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/mac/src/pe/lim/lim_process_assoc_req_frame.c b/core/mac/src/pe/lim/lim_process_assoc_req_frame.c index 3ecf3c17f6d2..2046f309a569 100644 --- a/core/mac/src/pe/lim/lim_process_assoc_req_frame.c +++ b/core/mac/src/pe/lim/lim_process_assoc_req_frame.c @@ -1893,7 +1893,7 @@ void lim_process_assoc_cleanup(struct mac_context *mac_ctx, qdf_mem_free(assoc_req); /* to avoid double free */ - if (assoc_req_copied && session->parsedAssocReq) + if (assoc_req_copied && session->parsedAssocReq && sta_ds) session->parsedAssocReq[sta_ds->assocId] = NULL; }