Amazing-Mirror/fragattacks
1
0
Fork 0
mirror of https://github.com/vanhoefm/fragattacks.git synced 2024-11-25 00:38:24 -05:00
Code Issues Projects Releases Wiki Activity

wlantest: Verify FTIE length before checking MIC

Browse Source 
tdls_verify_mic() and tdls_verify_mic_teardown() could have tried to
read the 16-octet FTIE MIC when processing a TDLS frame even if the
received FTIE is truncated. At least in theory, this could result in
reading couple of octets beyond the frame buffer.

Signed-off-by: Jouni Malinen <j@w1.fi>
This commit is contained in:
Jouni Malinen 2015-04-19 11:42:02 +03:00
parent 253103681c
commit d4c4ec9240
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
wlantest/rx_tdls.c
View File

@ -142,7 +142,8 @@ static int tdls_verify_mic(struct wlantest *wt, struct wlantest_tdls *tdls,
struct rsn_ftie *tmp_ftie; struct rsn_ftie *tmp_ftie;
if (elems->link_id == NULL || elems->rsn_ie == NULL || if (elems->link_id == NULL || elems->rsn_ie == NULL ||
elems->timeout_int == NULL || elems->ftie == NULL) elems->timeout_int == NULL || elems->ftie == NULL ||
elems->ftie_len < sizeof(struct rsn_ftie))
return -1; return -1;
len = 2 * ETH_ALEN + 1 + 2 + 18 + 2 + elems->rsn_ie_len + len = 2 * ETH_ALEN + 1 + 2 + 18 + 2 + elems->rsn_ie_len +
@ -488,7 +489,8 @@ static int tdls_verify_mic_teardown(struct wlantest *wt,
const struct rsn_ftie *rx_ftie; const struct rsn_ftie *rx_ftie;
struct rsn_ftie *tmp_ftie; struct rsn_ftie *tmp_ftie;
if (elems->link_id == NULL || elems->ftie == NULL) if (elems->link_id == NULL || elems->ftie == NULL ||
elems->ftie_len < sizeof(struct rsn_ftie))
return -1; return -1;
len = 2 + 18 + 2 + 1 + 1 + 2 + elems->ftie_len; len = 2 + 18 + 2 + 1 + 1 + 2 + elems->ftie_len;