samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

qcacld-3.0: refine the function for checking HE support

Browse Source 
Refine the function wlan_son_is_he_supported() to
fix a NULL pointer issue.

Change-Id: I8697162649dc06ac8f06dd2fb5fe819b712ede30
CRs-Fixed: 3189354
This commit is contained in:
Yu Wang
2022-05-05 14:59:51 +08:00
committed by Madan Koyyalamudi
parent cbccd75d42
commit 3052e9f536
1 changed files with 8 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

22
components/son/dispatcher/src/son_api.c
View File

@@ -69,26 +69,21 @@ wlan_son_register_mlme_deliver_cb(struct wlan_objmgr_psoc *psoc,
/** /**
* wlan_son_is_he_supported() - is he supported or not * wlan_son_is_he_supported() - is he supported or not
* @psoc: pointer to psoc * @psoc: pointer to psoc
* @he_supported: he supported or not
* *
* Return: void * Return: true if supports, false otherwise
*/ */
#ifdef WLAN_FEATURE_11AX #ifdef WLAN_FEATURE_11AX
static void wlan_son_is_he_supported(struct wlan_objmgr_psoc *psoc, static bool wlan_son_is_he_supported(struct wlan_objmgr_psoc *psoc)
bool *he_supported)
{ {
tDot11fIEhe_cap *he_cap = NULL; tDot11fIEhe_cap he_cap = {0};
*he_supported = false; mlme_cfg_get_he_caps(psoc, &he_cap);
mlme_cfg_get_he_caps(psoc, he_cap); return !!he_cap.present;
if (he_cap && he_cap->present)
*he_supported = true;
} }
#else #else
static void wlan_son_is_he_supported(struct wlan_objmgr_psoc *psoc, static bool wlan_son_is_he_supported(struct wlan_objmgr_psoc *psoc)
bool *he_supported)
{ {
*he_supported = false; return false;
} }
#endif /*WLAN_FEATURE_11AX*/ #endif /*WLAN_FEATURE_11AX*/
@@ -173,8 +168,7 @@ uint32_t wlan_son_get_chan_flag(struct wlan_objmgr_pdev *pdev,
return flags; return flags;
} }
wlan_son_is_he_supported(psoc, &is_he_enabled); is_he_enabled = wlan_son_is_he_supported(psoc);
wlan_mlme_get_sub_20_chan_width(wlan_pdev_get_psoc(pdev), wlan_mlme_get_sub_20_chan_width(wlan_pdev_get_psoc(pdev),
&sub_20_channel_width); &sub_20_channel_width);