From: Jes Sorensen <Jes.Sorensen@redhat.com>
Date: Tue, 15 Apr 2014 17:44:30 +0000 (+0200)
Subject: staging: rtl8723au: OnAction23a_back23a(): Use struct ieee8011_mgmt to parse action... 
X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=b30f74e0e06697e984c06b056769f52fc4354c0d;p=linux-beck.git

staging: rtl8723au: OnAction23a_back23a(): Use struct ieee8011_mgmt to parse action frames

Signed-off-by: Jes Sorensen <Jes.Sorensen@redhat.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
index c113fe3afe2b..ab6b00e2568d 100644
--- a/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
+++ b/drivers/staging/rtl8723au/core/rtw_mlme_ext.c
@@ -2000,7 +2000,6 @@ static int OnAction23a_back23a(struct rtw_adapter *padapter,
 	u8 *addr;
 	struct sta_info *psta = NULL;
 	struct recv_reorder_ctrl *preorder_ctrl;
-	unsigned char *frame_body;
 	unsigned char category, action;
 	unsigned short tid, status, capab, params, reason_code = 0;
 	struct mlme_ext_priv *pmlmeext = &padapter->mlmeextpriv;
@@ -2025,17 +2024,18 @@ static int OnAction23a_back23a(struct rtw_adapter *padapter,
 	if (!psta)
 		return _SUCCESS;
 
-	frame_body = &mgmt->u.action.category;
-
 	category = mgmt->u.action.category;
 	if (category == WLAN_CATEGORY_BACK) { /*  representing Block Ack */
 		if (!pmlmeinfo->HT_enable)
 			return _SUCCESS;
-		action = frame_body[1];
+		/* action_code is located in the same place for all
+		   action events, so pick any */
+		action = mgmt->u.action.u.wme_action.action_code;
 		DBG_8723A("%s, action =%d\n", __func__, action);
 		switch (action) {
 		case WLAN_ACTION_ADDBA_REQ: /* ADDBA request */
-			memcpy(&pmlmeinfo->ADDBA_req, &frame_body[2],
+			memcpy(&pmlmeinfo->ADDBA_req,
+			       &mgmt->u.action.u.addba_req.dialog_token,
 			       sizeof(struct ADDBA_request));
 			process_addba_req23a(padapter,
 					     (u8 *)&pmlmeinfo->ADDBA_req, addr);