aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* OWE: Try another group only on association rejection with status 77Ashok Kumar2018-12-023-2/+11
| | | | | | | | | Do not change the OWE group if association is rejected for any other reason than WLAN_STATUS_FINITE_CYCLIC_GROUP_NOT_SUPPORTED to avoid unnecessary latency in cases where the APs reject association, e.g., for load balancing reasons. Signed-off-by: Ashok Kumar <aponnaia@codeaurora.org>
* OWE: Fix association rejection behaviorJouni Malinen2018-12-022-3/+8
| | | | | | | | | | | If association failed for any non-OWE specific reason, the previous implementation tried to add the OWE related IEs into the (Re)Association Response frame. This is not needed and could actually result in dereferencing a NULL pointer. Fix this by adding those OWE related IEs only for successful association and only if the RSN state machine has been initialized. Signed-off-by: Jouni Malinen <j@w1.fi>
* nl80211: Debug print TX queue parameter values and resultJouni Malinen2018-12-021-1/+6
| | | | | | | | | | | | Some mac80211_hwsim test cases have failed with mysterious sequence where mac80211 has claimed the parameters are invalid ("wlan3: invalid CW_min/CW_max: 9484/40"). Those values look strange since they are not from hostapd configuration or default values.. hostapd is seeing TX queue parameter set failing for queues 0, 1, and 3 (but not 2) for these cases. Add debug prints to hostapd to get more details on what exactly is happening if such error cases can be reproduced. Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP network addition failureJouni Malinen2018-12-021-0/+46
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP and continue listen stateJouni Malinen2018-12-021-0/+29
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* DPP: Remove unused wpas_dpp_remain_on_channel_cb()Jouni Malinen2018-12-022-25/+0
| | | | | | This function was apparently never used at all. Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP_BOOTSTRAP_GEN/REMOVE/GET_URI/INFO error casesJouni Malinen2018-12-021-0/+51
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP and unknown configurator idJouni Malinen2018-12-021-0/+19
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP and PKEX with local failure in processing Commit Reveal ReqJouni Malinen2018-12-021-0/+33
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP and PKEX with local failure in processing Exchange RespJouni Malinen2018-12-021-0/+32
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP and PKEX with mismatching curve (local failure)Jouni Malinen2018-12-021-0/+45
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP PKEX identifier combinationsJouni Malinen2018-12-021-3/+39
| | | | | | Check behavior with valid and invalid identifier combinations. Signed-off-by: Jouni Malinen <j@w1.fi>
* DPP: Do not reply to PKEX request with identifier if no local identifierJouni Malinen2018-12-021-18/+31
| | | | | | | | | | | The reverse case (local identifier configured but no identifier received) was already covered, but PKEX is not going to complete successfully if there is any difference in identifier configuration, so ignore this other case as well. This avoids unnecessary responses to PKEX requests with identifier from a device that is ready for PKEX in general, but not for that particular request. Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: SAE and password identifier with FFCJouni Malinen2018-12-021-3/+18
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP peer introduction local failuresJouni Malinen2018-12-011-39/+126
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP peer introduction failuresJouni Malinen2018-12-011-0/+62
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* DPP: Apply testing configuration option to signing of own configJouni Malinen2018-12-012-0/+2
| | | | | | | Previous implementation had missed this case of setting configurator parameters. Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP own config signing failureJouni Malinen2018-12-011-0/+16
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP invalid configurator keyJouni Malinen2018-12-011-0/+23
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: DPP legacy parameters local failureJouni Malinen2018-12-011-0/+29
| | | | Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: Radio measurement capability with roamingJouni Malinen2018-12-012-0/+62
| | | | | | | This verifies that radio measurement capabilities are negotiated correctly for the reassociation cases with and without FT. Signed-off-by: Jouni Malinen <j@w1.fi>
* FT: Fix Reassociation Request IEs during FT protocolJouni Malinen2018-12-011-1/+79
| | | | | | | | | | | | | | | The previous implementation ended up replacing all pending IEs prepared for Association Request frame with the FT specific IEs (RSNE, MDE, FTE) when going through FT protocol reassociation with the wpa_supplicant SME. This resulted in dropping all other IEs that might have been prepared for the association (e.g., Extended Capabilities, RM Enabled Capabilities, Supported Operating Classes, vendor specific additions). Fix this by replacing only the known FT specific IEs with the appropriate values for FT protocol while maintaining other already prepared elements. Signed-off-by: Jouni Malinen <j@w1.fi>
* tests: Do not generate /tmp/hwsim-tests-*.tar.gz in VM caseJouni Malinen2018-12-011-1/+3
| | | | | | | There is no point in building this tarball in /tmp that is on the ramdisk of the VM since it will go away when the VM exits. Signed-off-by: Jouni Malinen <j@w1.fi>
* Use more consistent Action frame RX handling in both AP mode pathsJouni Malinen2018-12-012-6/+18
| | | | | | | | | | | | | Both handle_action() and hostapd_action_rx() are used for processing received Action frames depending on what type of driver architecture is used (MLME in hostapd vs. driver) and which build options were used to build hostapd. These functions had a bit different sequence for checking the frame and printing debug prints. Make those more consistent by checking that the frame includes the category-specific action field and some payload. Add a debug print for both functions to make it easier to see which path various Action frames use. Signed-off-by: Jouni Malinen <j@w1.fi>
* FT: Check session_timeout pointer consistentlyJouni Malinen2018-11-301-1/+1
| | | | | | | Avoid smatch warning on this even thought the only caller of the function uses a non-NULL pointer in all cases. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* Fix hostapd testing functionality for setting key/seqJouni Malinen2018-11-301-1/+1
| | | | | | Use sizeof() correctly on seq[]. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* EAP DB: Use NULL to clear a pointerJouni Malinen2018-11-301-1/+1
| | | | | | Avoid a sparse warning from use of a plain integer. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* DPP: Check own_bi/peer_bi pointers more consistentlyJouni Malinen2018-11-301-4/+4
| | | | | | This gets rid of smatch warnings about a dereference before check. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* Fix indentation levelJouni Malinen2018-11-307-31/+30
| | | | | | This gets rid of smatch warnings about inconsistent indenting. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* OpenSSL: Include sha512.h to match function prototypesJouni Malinen2018-11-301-0/+1
| | | | | | This gets rid of sparse warnings. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* macsec_linux: Make create_transmit_sc() handler use matching argumentsJouni Malinen2018-11-301-1/+1
| | | | | | | The currently unused conf_offset parameter used a mismatching type (enum vs. unsigned int) compared to the prototype. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* macsec_linux: Use NULL to clear a pointerJouni Malinen2018-11-301-1/+1
| | | | | | Avoid a sparse warning from use of a plain integer. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* macsec_qca: Mark macsec_qca_set_transmit_next_pn() staticJouni Malinen2018-11-301-1/+1
| | | | | | | This function is not used outside this C file. Mark it static to avoid a warning from sparse. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* SAE: Fix external authentication on big endian platformsAshok Ponnaiah2018-11-301-9/+10
| | | | | | | | Need to handle the little endian 16-bit fields properly when building and parsing Authentication frames. Fixes: 5ff39c1380d9 ("SAE: Support external authentication offload for driver-SME cases") Signed-off-by: Ashok Ponnaiah <aponnaia@codeaurora.org>
* tests: DPP invalid legacy parametersJouni Malinen2018-11-301-0/+56
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* DPP: Reject invalid no-psk/pass legacy configurator parametersJouni Malinen2018-11-302-26/+47
| | | | | | | | | | | Instead of going through the configuration exchange, reject invalid legacy configurator parameters explicitly. Previously, configuring legacy (psk/sae) parameters without psk/pass resulted in a config object that used a zero length passphrase. With this change, that config object is not sent and instead, either the initialization attempts is rejected or the incoming initialization attempt is ignored. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: DPP protocol testing - Auth Conf RX processing failureJouni Malinen2018-11-301-0/+20
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: DPP Auth Resp AES-SIV issueJouni Malinen2018-11-301-0/+33
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: More DPP incompatible roles coverageJouni Malinen2018-11-301-3/+47
| | | | | | Cover the Configurator/Configurator case in addition Enrollee/Enrollee. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: DPP and Auth Resp(status) build failureJouni Malinen2018-11-301-0/+6
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: DPP bootstrap key autogen issuesJouni Malinen2018-11-301-0/+38
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* OWE: Fix a compiler warning in non-testing buildJouni Malinen2018-11-271-1/+1
| | | | | | | | | | The new conf variable was used only within the CONFIG_TESTING_OPTIONS block and as such, added a warning about unused variable into non-testing builds. Fix that by using that variable outside the conditional block as well. Fixes: a22e235fd0df ("OWE: Add testing RSNE for OWE assoc response with driver SME/MLME") Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: DPP protocol testing cases for Auth Resp status-not-OK casesJouni Malinen2018-11-271-4/+47
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* DPP: Fix no-Status protocol testing in Auth Resp error caseJouni Malinen2018-11-271-1/+1
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: Additional DPP AES-SIV local failure coverageJouni Malinen2018-11-271-0/+1
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: DPP QR Code and keygen failureJouni Malinen2018-11-271-0/+12
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: DPP local failure on hashing public key for PKEX bootstrap infoJouni Malinen2018-11-271-1/+2
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: DPP QR Code and unsupported curveJouni Malinen2018-11-271-0/+15
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* Define QCA_NL80211_VENDOR_SUBCMD_LINK_PROPERTIES also as an eventSunil Dutt2018-11-261-0/+27
| | | | | | | | This commit enhances QCA_NL80211_VENDOR_SUBCMD_LINK_PROPERTIES to also be an event, aimed to notify the link status (EX: connected stations status on an AP link). Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
* tests: Additional DPP bootstrapping URI parsing coverageJouni Malinen2018-11-261-1/+30
| | | | Signed-off-by: Jouni Malinen <jouni@codeaurora.org>