From 92c8c2bb8482435675f4dc2ed196f46daddfa72b Mon Sep 17 00:00:00 2001
From: Wu Gao <wugao@codeaurora.org>
Date: Wed, 13 Oct 2021 04:17:50 +0800
Subject: [PATCH] qcacmn: Check function pointer to avoid NULL point
 dereference

It calls spectral_process_phyerr of spectral operations without any
check. Which cause null point dereference potentially. So, check this
pointer before calling.

Change-Id: Iae246f847dd41e62b628223652a9527a65b43799
CRs-Fixed: 3055246
---
 target_if/spectral/target_if_spectral.h | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/target_if/spectral/target_if_spectral.h b/target_if/spectral/target_if_spectral.h
index 1ff8609ee436..3add3240564c 100644
--- a/target_if/spectral/target_if_spectral.h
+++ b/target_if/spectral/target_if_spectral.h
@@ -1732,6 +1732,10 @@ void target_if_spectral_process_phyerr(
 	}
 
 	p_sops = GET_TARGET_IF_SPECTRAL_OPS(spectral);
+	if (!p_sops->spectral_process_phyerr) {
+		spectral_err("null spectral_process_phyerr");
+		return;
+	}
 	p_sops->spectral_process_phyerr(spectral, data, datalen,
 					p_rfqual, p_chaninfo,
 					tsf64, acs_stats);