diff options
| author | Sunil Dutt <usdutt@qti.qualcomm.com> | 2015-10-20 09:50:51 +0530 |
|---|---|---|
| committer | Gerrit - the friendly Code Review server <code-review@localhost> | 2015-10-29 04:35:25 -0700 |
| commit | 5556b4903366e6ffacd0d99e855d15df95108596 (patch) | |
| tree | 68651ac81d46db221a56e77323cac08bd337f0fb /src | |
| parent | 2c2fb0b85d7fef10a2827a64724e79583c70995a (diff) | |
| download | android_external_wpa_supplicant_8-5556b4903366e6ffacd0d99e855d15df95108596.tar.gz android_external_wpa_supplicant_8-5556b4903366e6ffacd0d99e855d15df95108596.tar.bz2 android_external_wpa_supplicant_8-5556b4903366e6ffacd0d99e855d15df95108596.zip | |
TDLS: Do not send error case of TPK M3 if TX fails
There is no point in sending TPK M3 (TDLS Setup Confirm) with a failure
status if the first transmission attempt fails. Instead, just return a
failure by disabling the link rather than retransmitting the TPK M3
frame with an error status.
CRs-Fixed: 930704
Change-Id: Id592ee93b2264cb10e934a6ebb477f8d580284cb
Git-commit: f67d1a009963e9d7c3071950e77237a9d290e4eb
Git-repo : git://w1.fi/srv/git/hostap.git
Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
Diffstat (limited to 'src')
| -rw-r--r-- | src/rsn_supp/tdls.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/rsn_supp/tdls.c b/src/rsn_supp/tdls.c index 2bbcab16..0ac04227 100644 --- a/src/rsn_supp/tdls.c +++ b/src/rsn_supp/tdls.c @@ -2391,7 +2391,7 @@ skip_rsn: wpa_printf(MSG_DEBUG, "TDLS: Sending TDLS Setup Confirm / " "TPK Handshake Message 3"); if (wpa_tdls_send_tpk_m3(sm, src_addr, dtoken, lnkid, peer) < 0) - goto error; + goto error_no_msg; if (!peer->tpk_success) { /* @@ -2412,6 +2412,7 @@ skip_rsn: error: wpa_tdls_send_error(sm, src_addr, WLAN_TDLS_SETUP_CONFIRM, dtoken, 1, status); +error_no_msg: wpa_tdls_disable_peer_link(sm, peer); return -1; } |