qtnfmac: fix invalid channel information output
authorSergey Matyukevich <sergey.matyukevich.os@quantenna.com>
Wed, 13 Nov 2019 11:06:51 +0000 (11:06 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 31 Dec 2019 15:45:27 +0000 (16:45 +0100)
[ Upstream commit 24227a9e956a7c9913a7e6e7199a9ae3f540fe88 ]

Do not attempt to print frequency for an invalid channel
provided by firmware. That channel may simply not exist.

Signed-off-by: Sergey Matyukevich <sergey.matyukevich.os@quantenna.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/net/wireless/quantenna/qtnfmac/event.c

index b57c8c1..7846383 100644 (file)
@@ -171,8 +171,9 @@ qtnf_event_handle_bss_join(struct qtnf_vif *vif,
                return -EPROTO;
        }
 
-       pr_debug("VIF%u.%u: BSSID:%pM status:%u\n",
-                vif->mac->macid, vif->vifid, join_info->bssid, status);
+       pr_debug("VIF%u.%u: BSSID:%pM chan:%u status:%u\n",
+                vif->mac->macid, vif->vifid, join_info->bssid,
+                le16_to_cpu(join_info->chan.chan.center_freq), status);
 
        if (status != WLAN_STATUS_SUCCESS)
                goto done;
@@ -181,7 +182,7 @@ qtnf_event_handle_bss_join(struct qtnf_vif *vif,
        if (!cfg80211_chandef_valid(&chandef)) {
                pr_warn("MAC%u.%u: bad channel freq=%u cf1=%u cf2=%u bw=%u\n",
                        vif->mac->macid, vif->vifid,
-                       chandef.chan->center_freq,
+                       chandef.chan ? chandef.chan->center_freq : 0,
                        chandef.center_freq1,
                        chandef.center_freq2,
                        chandef.width);