Messages in this thread Patch in this message |  | | From | Martin Kaiser <> | Subject | [PATCH 06/11] staging: r8188eu: don't copy ra and ta before we fail | Date | Sun, 3 Apr 2022 18:54:33 +0200 |
| |
In validate_recv_data_frame, we return an error if both to_ds and from_ds are set in the incoming data frame. There's no need to populate patrib->ra and ta before we return. The caller will free the received frame, including pattrib.
Signed-off-by: Martin Kaiser <martin@kaiser.cx> --- drivers/staging/r8188eu/core/rtw_recv.c | 5 ----- 1 file changed, 5 deletions(-)
diff --git a/drivers/staging/r8188eu/core/rtw_recv.c b/drivers/staging/r8188eu/core/rtw_recv.c index 47d4fd01824f..c200e1251545 100644 --- a/drivers/staging/r8188eu/core/rtw_recv.c +++ b/drivers/staging/r8188eu/core/rtw_recv.c @@ -974,11 +974,6 @@ static int validate_recv_data_frame(struct adapter *adapter, memcpy(pattrib->ta, psa, ETH_ALEN); ret = sta2ap_data_frame(adapter, precv_frame, &psta); break; - case 3: - memcpy(pattrib->ra, GetAddr1Ptr(ptr), ETH_ALEN); - memcpy(pattrib->ta, GetAddr2Ptr(ptr), ETH_ALEN); - ret = _FAIL; - break; default: ret = _FAIL; break; -- 2.30.2
|  |