[PATCH 2/3] staging: Dynamically allocate struct station_info

[Toke Høiland-Jørgensen]

Since the addition of the TXQ stats to cfg80211, the station_info struct
has grown to be quite large, which results in warnings when allocated on
the stack. Fix the affected places to do dynamic allocations instead.
WARN_ON is used where the function has no way to signal errors to the
caller.

This patch applies the fix to the rtl8723bs driver in staging while a
separate patch fixes the drivers in the main tree.

Fixes: 52539ca89f36 cfg80211: Expose TXQ stats and parameters to userspace
Signed-off-by: Toke Høiland-Jørgensen <t...@toke.dk>
---
 drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c |   16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c 
b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
index 46bc2e512557..c76c9a8066c4 100644
--- a/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
+++ b/drivers/staging/rtl8723bs/os_dep/ioctl_cfg80211.c
@@ -2431,17 +2431,23 @@ void rtw_cfg80211_indicate_sta_assoc(struct adapter 
*padapter, u8 *pmgmt_frame,
        DBG_871X(FUNC_ADPT_FMT"\n", FUNC_ADPT_ARG(padapter));
 
        {
-               struct station_info sinfo;
+               struct station_info *sinfo;
                u8 ie_offset;
                if (GetFrameSubType(pmgmt_frame) == WIFI_ASSOCREQ)
                        ie_offset = _ASOCREQ_IE_OFFSET_;
                else /*  WIFI_REASSOCREQ */
                        ie_offset = _REASOCREQ_IE_OFFSET_;
 
-               sinfo.filled = 0;
-               sinfo.assoc_req_ies = pmgmt_frame + WLAN_HDR_A3_LEN + ie_offset;
-               sinfo.assoc_req_ies_len = frame_len - WLAN_HDR_A3_LEN - 
ie_offset;
-               cfg80211_new_sta(ndev, GetAddr2Ptr(pmgmt_frame), &sinfo, 
GFP_ATOMIC);
+               sinfo = kzalloc(sizeof(*sinfo), GFP_KERNEL);
+               if (WARN_ON(!sinfo))
+                       return;
+
+               sinfo->filled = 0;
+               sinfo->assoc_req_ies = pmgmt_frame + WLAN_HDR_A3_LEN + 
ie_offset;
+               sinfo->assoc_req_ies_len = frame_len - WLAN_HDR_A3_LEN - 
ie_offset;
+               cfg80211_new_sta(ndev, GetAddr2Ptr(pmgmt_frame), sinfo, 
GFP_ATOMIC);
+
+               kfree(sinfo);
        }
 }