[lvc-project] [PATCH] brcmfmac: remove redundant NULL check in brcmf_run_escan()

[Dmitriy Okunev]

The 'request' pointer is guaranteed to be non-NULL at this point,
so the check is unnecessary. No functional change, just code cleanup.

Found by Linux Verification Center (linuxtesting.org) with SVACE.

Signed-off-by: Dmitriy Okunev <dmitriy.okunev at red-soft.ru>
---
 .../wireless/broadcom/brcm80211/brcmfmac/cfg80211.c    | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
index 89f61710a210..46114fa47c3a 100644
--- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
+++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c
@@ -1450,13 +1450,11 @@ brcmf_run_escan(struct brcmf_cfg80211_info *cfg, struct brcmf_if *ifp,
 
 	brcmf_dbg(SCAN, "E-SCAN START\n");
 
-	if (request != NULL) {
-		/* Allocate space for populating ssids in struct */
-		params_size += sizeof(u32) * ((request->n_channels + 1) / 2);
+	/* Allocate space for populating ssids in struct */
+	params_size += sizeof(u32) * ((request->n_channels + 1) / 2);
 
-		/* Allocate space for populating ssids in struct */
-		params_size += sizeof(struct brcmf_ssid_le) * request->n_ssids;
-	}
+	/* Allocate space for populating ssids in struct */
+	params_size += sizeof(struct brcmf_ssid_le) * request->n_ssids;
 
 	params = kzalloc(params_size, GFP_KERNEL);
 	if (!params) {
-- 
2.53.0