Commit 7711d231 authored May 19, 2017 by Gustavo A. R. Silva Committed by Gerrit - the friendly Code Review server Jun 15, 2020

ath10k: remove unnecessary code

The array fields in struct wmi_start_scan_arg that are checked here are
fixed size arrays so they can never be NULL.

Change-Id: I5c80a9758328df2eff1fa7c4bb433d0370f67143
Addresses-Coverity-ID: 1260031
Cc: Arend Van Spriel <arend.vanspriel@broadcom.com>
Cc: Kalle Valo <kvalo@qca.qualcomm.com>
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Signed-off-by: Kalle Valo <kvalo@qca.qualcomm.com>
Git-repo: https://source.codeaurora.org/quic/la/kernel/msm-4.4

Git-commit: 8c1f20815231edf1ed2b7133ea07ea94ca31db05
Signed-off-by: Swetha Chikkaboraiah <schikk@codeaurora.org>

parent 5bd445d8

drivers/net/wireless/ath/ath10k/wmi.c

+0 −9

Original line number	Diff line number	Diff line
		@@ -6048,15 +6048,6 @@ static struct sk_buff ath10k_wmi_10_4_op_gen_init(struct ath10k ar)

		int ath10k_wmi_start_scan_verify(const struct wmi_start_scan_arg *arg)
		{
		if (arg->ie_len)
		return -EINVAL;
		if (arg->n_channels && !arg->channels)
		return -EINVAL;
		if (arg->n_ssids && !arg->ssids)
		return -EINVAL;
		if (arg->n_bssids && !arg->bssids)
		return -EINVAL;

		if (arg->ie_len > WLAN_SCAN_PARAMS_MAX_IE_LEN)
		return -EINVAL;
		if (arg->n_channels > ARRAY_SIZE(arg->channels))