Commit 19be9e9a authored by Maninder Singh's avatar Maninder Singh Committed by Kalle Valo

ath10k: fix wrong initialization of struct channel

chandef is initialized with NULL and on the very next line, we are using it to
get channel, which is not correct. Channel should be initialized after
obtaining chandef.

Found by cppcheck:

ath/ath10k/mac.c:839]: (error) Possible null pointer dereference: chandef
Signed-off-by: default avatarManinder Singh <maninder1.s@samsung.com>
Signed-off-by: default avatarKalle Valo <kvalo@qca.qualcomm.com>
parent 1f5dbfbb
...@@ -836,7 +836,7 @@ static inline int ath10k_vdev_setup_sync(struct ath10k *ar) ...@@ -836,7 +836,7 @@ static inline int ath10k_vdev_setup_sync(struct ath10k *ar)
static int ath10k_monitor_vdev_start(struct ath10k *ar, int vdev_id) static int ath10k_monitor_vdev_start(struct ath10k *ar, int vdev_id)
{ {
struct cfg80211_chan_def *chandef = NULL; struct cfg80211_chan_def *chandef = NULL;
struct ieee80211_channel *channel = chandef->chan; struct ieee80211_channel *channel = NULL;
struct wmi_vdev_start_request_arg arg = {}; struct wmi_vdev_start_request_arg arg = {};
int ret = 0; int ret = 0;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment