The patch below does not apply to the 5.4-stable tree. If someone wants it applied there, or to any other stable or longterm tree, then please email the backport, including the original git commit id to stable@vger.kernel.org.
To reproduce the conflict and resubmit, you may use the following commands:
git fetch https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/ linux-5.4.y git checkout FETCH_HEAD git cherry-pick -x d17789edd6a8270c38459e592ee536a84c6202db # <resolve conflicts, build, test, etc.> git commit -s git send-email --to 'stable@vger.kernel.org' --in-reply-to '167870428622493@kroah.com' --subject-prefix 'PATCH 5.4.y' HEAD^..
Possible dependencies:
d17789edd6a8 ("staging: rtl8723bs: Pass correct parameters to cfg80211_get_bss()") 6994aa430368 ("staging: rtl8723bs: fix camel case in struct ndis_802_11_ssid") d8b322b60da6 ("staging: rtl8723bs: fix camel case in struct ndis_802_11_conf") d3fcee1b78a5 ("staging: rtl8723bs: fix camel case in struct wlan_bssid_ex") 2a62ff13132a ("staging: rtl8723bs: remove commented out condition") ddd7c8b0033b ("staging: rtl8723bs: remove 5Ghz code blocks") ec84d0ae54a6 ("staging: rtl8723bs: remove 5Ghz code related to channel plan definition") 708180a92cd8 ("staging: rtl8723bs: remove all 5Ghz network types") 403e6946d119 ("staging: rtl8723bs: remove if (true) statement") 2172a6576388 ("staging: rtl8723bs: remove commented out RT_ASSERT occurrences") 98dc120895a9 ("staging: rtl8723bs: split too long line") 5a94f5c84281 ("staging: rtl8723bs: fix indentation in if block") 79df841b4350 ("staging: rtl8723bs: replace DBG_871X_LEVEL logs with netdev_*()") 027ffa10b80b ("staging: rtl8723bs: remove sdio_drv_priv structure") ee31d57e4022 ("staging: rtl8723bs: remove unnecessary dump_drv_version() usage") a2e2a05d5d57 ("staging: rtl8723bs: remove unnecessary goto jumps") 5ffbfcf38b80 ("staging: rtl8723bs: remove empty ifdef blocks conditioned to DEBUG_CFG80211 definition") dcc48e083749 ("staging: rtl8723bs: remove all DBG_8192C logs") af6afdb63f17 ("staging: rtl8723bs: split long lines") e427bdd8e1e5 ("staging: rtl8723bs: rewrite comparison to null")
thanks,
greg k-h
------------------ original commit in Linus's tree ------------------
From d17789edd6a8270c38459e592ee536a84c6202db Mon Sep 17 00:00:00 2001 From: Hans de Goede hdegoede@redhat.com Date: Mon, 6 Mar 2023 16:35:12 +0100 Subject: [PATCH] staging: rtl8723bs: Pass correct parameters to cfg80211_get_bss()
To last 2 parameters to cfg80211_get_bss() should be of the enum ieee80211_bss_type resp. enum ieee80211_privacy types, which WLAN_CAPABILITY_ESS very much is not.
Fix both cfg80211_get_bss() calls in ioctl_cfg80211.c to pass the right parameters.
Note that the second call was already somewhat fixed by commenting out WLAN_CAPABILITY_ESS and passing in 0 instead. This was still not entirely correct though since that would limit returned BSS-es to ESS type BSS-es with privacy on.
Cc: stable@vger.kernel.org Signed-off-by: Hans de Goede hdegoede@redhat.com Link: https://lore.kernel.org/r/20230306153512.162104-2-hdegoede@redhat.com Signed-off-by: Greg Kroah-Hartman gregkh@linuxfoundation.org
diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c index 3aba4e6eec8a..84a9f4dd8f95 100644 --- a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c +++ b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c @@ -350,7 +350,7 @@ int rtw_cfg80211_check_bss(struct adapter *padapter) bss = cfg80211_get_bss(padapter->rtw_wdev->wiphy, notify_channel, pnetwork->mac_address, pnetwork->ssid.ssid, pnetwork->ssid.ssid_length, - WLAN_CAPABILITY_ESS, WLAN_CAPABILITY_ESS); + IEEE80211_BSS_TYPE_ANY, IEEE80211_PRIVACY_ANY);
cfg80211_put_bss(padapter->rtw_wdev->wiphy, bss);
@@ -1139,8 +1139,8 @@ void rtw_cfg80211_unlink_bss(struct adapter *padapter, struct wlan_network *pnet
bss = cfg80211_get_bss(wiphy, NULL/*notify_channel*/, select_network->mac_address, select_network->ssid.ssid, - select_network->ssid.ssid_length, 0/*WLAN_CAPABILITY_ESS*/, - 0/*WLAN_CAPABILITY_ESS*/); + select_network->ssid.ssid_length, IEEE80211_BSS_TYPE_ANY, + IEEE80211_PRIVACY_ANY);
if (bss) { cfg80211_unlink_bss(wiphy, bss);