From e4430b559d75e73a513da3c7ed66e797ae41b704 Mon Sep 17 00:00:00 2001 From: Sandeep Puligilla Date: Wed, 2 May 2018 18:22:21 -0700 Subject: [PATCH] qcacmn: Add NULL validation check in tgt_scan_start() API Add NULL validation check for psoc, pdev, vdev object pointers in tgt_scan_start() API. Change-Id: Ia5b6e5de26e4ad3386fbcef2fa1a5abb52198a1b CRs-Fixed: 2232783 --- umac/scan/dispatcher/src/wlan_scan_tgt_api.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/umac/scan/dispatcher/src/wlan_scan_tgt_api.c b/umac/scan/dispatcher/src/wlan_scan_tgt_api.c index b2a84d29354f..448cab52a7af 100644 --- a/umac/scan/dispatcher/src/wlan_scan_tgt_api.c +++ b/umac/scan/dispatcher/src/wlan_scan_tgt_api.c @@ -128,8 +128,17 @@ tgt_scan_start(struct scan_start_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_start_cmd_send() */