From 008d66251dcafabc64fd46bc6f8ea470bae19a52 Mon Sep 17 00:00:00 2001 From: Sandeep Puligilla Date: Wed, 2 May 2018 18:24:21 -0700 Subject: [PATCH] qcacmn: Add NULL validation check in tgt_scan_cancel() API Add NULL validation check for psoc, pdev, vdev object pointers in tgt_scan_cancel() API Change-Id: I7853d8bba439d40a1413b951d8aa799c932b887c CRs-Fixed: 2232802 --- umac/scan/dispatcher/src/wlan_scan_tgt_api.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/umac/scan/dispatcher/src/wlan_scan_tgt_api.c b/umac/scan/dispatcher/src/wlan_scan_tgt_api.c index 448cab52a7af..4321f6ed27d7 100644 --- a/umac/scan/dispatcher/src/wlan_scan_tgt_api.c +++ b/umac/scan/dispatcher/src/wlan_scan_tgt_api.c @@ -158,9 +158,16 @@ tgt_scan_cancel(struct scan_cancel_request *req) struct wlan_objmgr_pdev *pdev; struct wlan_objmgr_vdev *vdev = req->vdev; + if (!vdev) { + scm_err("vdev is NULL"); + return QDF_STATUS_E_NULL_VALUE; + } psoc = wlan_vdev_get_psoc(vdev); pdev = wlan_vdev_get_pdev(vdev); - + if (!psoc || !pdev) { + scm_err("psoc: 0x%pK or pdev: 0x%pK is NULL", psoc, pdev); + return QDF_STATUS_E_NULL_VALUE; + } scan_ops = wlan_psoc_get_scan_txops(psoc); /* invoke wmi_unified_scan_stop_cmd_send() */ QDF_ASSERT(scan_ops->scan_cancel);