Interworking: Remove unused allow_excluded from interworking_connect()

This argument was set to a hardcoded value 1 and as such, the other
cases were not reachable.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 connection and roaming consortium constraints

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 network selection with req_conn_capab (not present)

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: D-Bus GetAll wpa_config_get_all() OOM

This is a regression test for a crash on wpa_config_get_all() error
path.

Signed-off-by: Jouni Malinen <j@w1.fi>

Fix wpa_config_get_all() error path

The previous version did not really work at all and it ended up crashing
if the os_strdup(field->name) call failed.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 OSU provider and no AP with info

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: GAS/ANQP query OOM

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: ANQP_GET to unknown BSS

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: GAS_REQUEST OOM

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 network selection and invalid GAS response

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 and maximum BSS load (roaming)

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 network selection with min bandwidth - special cases

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 required roaming consortium and no match

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 connection and invalid roaming consortium ANQP-element

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 with simulated SIM and EAP-SIM - OOM

Signed-off-by: Jouni Malinen <j@w1.fi>

Interworking: Combine identical error returns into a single one

There is no need to maintain separate "return -1;" for each of the
wpa_config_set() calls that cannot really fail in practice.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Fix interworking_select() bssid argument in some test cases

The bssid argument was missing from couple of test cases. While this is
clearly incorrect, it looks like the error did not really cause any
significant issues to the test case. Anyway, better provide the correct
set of arguments to the call.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 with simulated SIM and EAP-SIM - invalid IMSI

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Remove extra semicolons from python scripts

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 network selection and OOM

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 network selection and no roaming consortium

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 network selection and cred_with_nai_realm cred->realm

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Hotspot 2.0 multi-cred and same sp_priority

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: wpa_supplicant P2P_LO_START and P2P_LO_STOP commands

Signed-off-by: Jouni Malinen <j@w1.fi>

P2P: Allow P2P listen being offloaded to the driver/firmware

This allows P2P Listen to be offloaded to device to enhance power
saving.

To start P2P listen offload, from wpa_cli interface, issue the command:
p2p_lo_start <freq> <period> <interval> <count>

To stop P2P listen offload, issue the command:
p2p_lo_stop

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>

nl80211: P2P Listen offload vendor command definitions

Define QCA vendor commands, events, and attributes for P2P
Listen offload.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>

Do not exceed scan ssid max size advertised by driver

Previously, wpa_set_scan_ssids() fully exhausted
wpa_driver_scan_params.ssid list when hidden network IDs are provided
via the control interface. This results in us exceeding the max size for
the list advertised by the driver when we add the "wildcard" scan SSID
entry. So, ensure that we leave space for one more scan SSID entry in
the list when we exit out of wpa_set_scan_ssids().

Signed-off-by: Roshan Pius <rpius@google.com>

tests: IBSS and key_mgmt field in STATUS

Signed-off-by: Jouni Malinen <j@w1.fi>

Add text name for WPA_KEY_MGMT_WPA_NONE key_mgmt value

This will output WPA-NONE for WPA_KEY_MGMT_WPA_NONE key_mgmt value in
STATUS command.

Signed-off-by: Saurav Babu <saurav.babu@samsung.com>

tests: IBSS operating frequency reporting

Signed-off-by: Jouni Malinen <j@w1.fi>

nl80211: Provide frequency in EVENT_ASSOC when IBSS is joined

Provides operating frequency in EVENT_ASSOC when IBSS is joined so that
wpa_s->assoc_freq can be updated when any IBSS network is joined.

Signed-off-by: Saurav Babu <saurav.babu@samsung.com>

tests: MBO and supported operating classes

Signed-off-by: Jouni Malinen <j@w1.fi>

MBO: Improve supported operating class generation

Previously, 2.4 GHz operating class 81 was not added for US due to not
all of the channels (1-13 in this operating class) being supported.
Still, this operating class is the main operating class in the global
table for 2.4 GHz and it is the only option for indicating support for
the 2.4 GHz band channels in US.

Change the supported operating class building rules to include all
operating classes for which at least one channel is enabled. In
addition, fix the 80, 80+80, and 160 MHz channel checks (checking the
center frequency channel was failing since it is not a valid 20 MHz
channel).

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: MBO BSS transition request MBO IE parsing

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: MBO failure cases

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: MBO WNM token wrap around

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Invalid MBO non_pref_chan values

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: MBO mbo_cell_capa duplicate update

Signed-off-by: Jouni Malinen <j@w1.fi>

nl80211: Keep QCA vendor extensions together

Signed-off-by: Dmitry Shmidt <dimitrysh@google.com>

tests: MPM protocol testing - CLS_ACPT event in OPN_SNT

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: MPM protocol testing - CLS_ACPT event in CNF_RCVD

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Mesh MPM FSM and HOLDING state event OPN_ACPT

Signed-off-by: Jouni Malinen <j@w1.fi>

mesh: Add debug prints on my/peer lid mismatches

This makes it easier to figure out why a received mesh peering frame
could end up getting dropped.

Signed-off-by: Jouni Malinen <j@w1.fi>

mesh: Add MPM FSM transitions from ESTAB to HOLDING for {OPN,CNF}_RJCT

These events were missing from the MPM FSM state transition table.

Signed-off-by: Jouni Malinen <j@w1.fi>

mesh: Update MPM FSM events to match the standard

{OPN,CNF,CLS}_IGNR events were removed during P802.11s draft development
process. Replace these with not generating a MPM FSM event. In addition,
add the standard REQ_RJCT event and option to pass in a specific reason
code to mesh_mpm_fsm().

Signed-off-by: Jouni Malinen <j@w1.fi>

mesh: Rename MPM FSM states to match the standard

During the P802.11s draft development, there were separate LISTEN and
IDLE states. However, the current IEEE 802.11 standards uses only the
IDLE state while the implementation called this LISTEN. Rename the state
in the implementation to match the one used in the standard to avoid
confusion. In addition, rename OPEN_{SENT,RCVD} to OPN_{SNT,RCVD} to
match the exact spelling of these states in the standard.

Signed-off-by: Jouni Malinen <j@w1.fi>

mesh: Indicate OPN_RJCT event if AES-SIV decrypt fails

REVmc/D6.0 14.5.5.2.2 (Processing Mesh Peering Open frames for AMPE)
mandates the OPN_RJCT event to be invoked if AES-SIV decryption for
received Mesh Peering Open frame fails. This allows a Mesh Peering Close
frame to be sent in such a case.

Signed-off-by: Jouni Malinen <j@w1.fi>

mesh: Write close reason from Mesh Peering Close to debug log

This makes it a bit easier to understand the debug log entries related
to tearing down a mesh connection.

Signed-off-by: Jouni Malinen <j@w1.fi>

mesh: Remove GTKdata and IGTKdata from Mesh Peering Confirm/Close

These optional fields are supposed to be included in the Authenticated
Mesh Peering Exchange element only in Mesh Peering Open frames.
Previously, these were incorrectly included in Mesh Peering
Confirm/Close frames and also required to be present in all these
frames.

While this commit changes the receive processing to ignore the
unexpected extra fields, it should be noted that the previous
implementation required the fields to be present and as such, the fixed
implementation is not compatible with it for secure mesh.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Secure mesh network connectivity failure

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Mesh peering management protocol testing for peer addition

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Open mesh network connectivity, no_auto on both peers

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Add TEST_FAIL() to hostapd_get_aid()

This allows additional testing coverage for AID unavailability cases.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Mark 525 tests as remote compatible

After successfully passing the 525 tests on a remote setup mark the
tests as remote compatible.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Add general filter for remote tests execution

Add a new function decorator for the test functions so that they can be
marked as remote compatible tests. Add a general filter to the remote
tests execution script to only execute tests that are remote compatible.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use general cmd_execute() for bridge setup commands

The hwsim tests used to execute shell commands in the tests using the
subprocess python module. Use the cmd_execute() general function for
executing shell commands to setup bridge so that this would also work on
remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() for iw connect command

The hwsim tests used to execute shell commands in the tests using the
subprocess python module. Use the cmd_execute() general function for
executing "iw connect ..." commands so that this would also work on
remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() for ip link set up/down commands

The hwsim tests used to execute shell commands in the tests using the
subprocess python module. Use the cmd_execute() general function for
executing "ip link set up/down" commands so that this would also work on
remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() for iw station get command

The hwsim tests used to execute shell commands in the tests using the
subprocess python module. Use the cmd_execute() general function for
executing "iw ... station get" commands so that this would also work on
remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() in find_wpas_process()

The hwsim tests used to execute shell commands in the tests using the
subprocess python module. Use the cmd_execute() general function for
executing "ps ax" so that this would also work on remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() for ip addr add/del

The hwsim tests used to execute shell commands in the tests using the
subprocess python module. Use the cmd_execute() general function for
executing "ip addr add/del .." so that this would also work on remote
setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() for iw scan

The hwsim tests used to execute shell commands in the tests using the
subprocess python module. Use the cmd_execute() general function for
executing "iw scan .." so that this would also work on remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() for iw reg set 00

The hwsim tests used to execute shell commands in the tests using the
subprocess python module. Use the cmd_execute() general function for
executing "iw reg set 00" so that this would also work on remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Remove unused import subprocess

Number of files imported subprocess without using anything from subprocess.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Add the forgotten wlantest_setup() call to a TDLS test

One TDLS test case was using wlantest without doing the setup first.
This makes the test not work on real hardware. Fix the issue by adding
the wlantest setup to the test.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() in set_powersave()

hwsim_utils.set_powersace() used to do file operations locally in
python. Start using the cmd_execute() general function for file
operations so that this would also work on remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Wait for CTRL-EVENT-CHANNEL-SWITCH event in ap_csa tests

This verifies that the channel switch is reported by the station and
replaces the fixed sleep before a traffic test by wait for the actual
switch operation to complete.

Signed-off-by: Jouni Malinen <j@w1.fi>

Add CTRL-EVENT-CHANNEL-SWITCH event to indicate channel changes

This provides information of the channel switch to wpa_supplicant
control interface monitors.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Use shell in local cmd_execute() only if needed

The generic cmd_execute() function was introduced in a manner that
converted the argument array to a string and used shell to run the
command unconditionally. This is not really desirable, so move back to
using the command array by default and use the single command string
with a shell only when really needed.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: Use cmd_execute() in ap_ht clear_scan_cache()

The ap_ht tests used to execute shell commands in the tests using the
subprocess python module. Complete the move to using the cmd_execute()
general function for executing shell commands so that this would also
work on remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use cmd_execute() function in ap_ht40_scan_not_affected

The ap_ht tests used to execute shell commands in the tests using the
subprocess python module. Start using the cmd_execute() general function
for executing shell commands so that this would also work on remote
setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use the cmd_execute() function in ap_ht for iw reg set

The ap_ht tests used to execute iw reg set command using the subprocess
python module. Start using the cmd_execute() general function for
executing shell commands so that this would also work on remote setups.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Use apdev[0] more consistently in ap_ht

The test cases ap_ht40_5ghz_invalid_pair and ap_ht40_5ghz_disabled_sec
mixed use of apdev[0] and apdev[1] while only needing a single AP. This
works when both the devices are on the same host (e.g., with
mac80211_hwsim), but not when using separate remote hosts. Fix this by
using apdev[0] more consistently in these test cases.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Remove trailing whitespace

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Pass full apdev to clear_scan_cache()

This is a step towards enabling the needed commands to be run on a
remote host.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: Initialize hapd variable before try-finally uses

Some TDLS and WPS test cases reference the hapd variable in the finally
block even if the test failed before assigning the value to this
variable. This makes the code in the finally block to fail on
referencing this variable. Assign None to the hapd variable before
starting the tests to avoid this.

Signed-off-by: Jonathan Afek <jonathanx.afek@intel.com>

tests: GAS failure cases

Signed-off-by: Jouni Malinen <j@w1.fi>

GAS: Fix double-free on an error path

If radio_add_work() fails, gas_query_req() ended up freeing the query
payload and returning an error. This resulted in also the caller trying
to free the query payload. Fix this by not freeing the buffer within
gas_query_req() in error case to be consistent with the other error
cases.

Signed-off-by: Jouni Malinen <j@w1.fi>

GAS: Remove unused gas_query_cancel()

This function was added with the initial GAS implementation, but there
was no user for it at the time and no clear use now either, so remove
the unused function and the related GAS query reason code.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: GAS/ANQP query protected vs. not protected

Signed-off-by: Jouni Malinen <j@w1.fi>

GAS: Check protected/unprotected drop after action code check

Apply the GAS specific rule of using Protected Dual of Public Action
frame only after having checked that the action code indicates this to
be a GAS response. Previously, non-GAS Public Action frames could have
been incorrectly dropped because of this check if received during an
association with PMF enabled.

Signed-off-by: Jouni Malinen <j@w1.fi>

GAS: Report GAS-QUERY-DONE event on initial req TX failure

The GAS-QUERY-DONE event with result=INTERNAL_ERROR was reported on most
other error cases, but the failure triggered by not being able to
transmit a GAS Initial Request stopped the query silently. Make this
more consistent with other error cases by reporting the same event.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: GAS query deleted at deinit

Signed-off-by: Jouni Malinen <j@w1.fi>

OpenSSL: Update to match the modified DH_get0_key() API

OpenSSL 1.1.0 (master branch) apparently ended up modifying the API
after the beta 2 release that was supposed to complete the work. Mark
the variables const to fix the compilation with the modified OpenSSL
API.

Signed-off-by: Jouni Malinen <j@w1.fi>

D-Bus: Make the CurrentAuthMode property getter more robust

Now that mesh sets wpa_s->wpa_state = WPA_COMPLETED, it looks like this
getter can get called with wpa_s->current_ssid = NULL. That could result
in NULL pointer dereference, so need to protect that specific case.

Signed-off-by: Jouni Malinen <j@w1.fi>

tests: D-Bus DeviceFoundProperties signal

Signed-off-by: Jouni Malinen <j@w1.fi>

D-Bus: Add DeviceFoundProperties signal for discovered peers

This signal sends the peer properties to applications for discovered
peers. The signature of this event is "oa{sv}". This event is needed
because the current DeviceFound signal provides only the peer object
path. If there are many peers in range there will be many DeviceFound
signals and for each DeviceFound signal, applications would need to use
GetAll to fetch peer properties. Doing this many times would create
extra load over application as well as over wpa_supplicant, so it is
better to send peer properties in the event so that applications can
extract found peer information without extra steps.

The existing DeviceFound signal is left as-is to avoid changing its
signature.

The issue is not applicable to the control interface because the
P2P-DEVICE-FOUND event includes peer info in it, but over D-Bus
interface DeviceFound provides only the peer object.

Signed-off-by: Nishant Chaprana <n.chaprana@samsung.com>

tests: wpa_state == COMPLETED in mesh

Signed-off-by: Jouni Malinen <j@w1.fi>

mesh: Mark wpa_state COMPLETED when mesh join has been performed

In mesh interface, the wpa_supplicant state was either
DISCONNECT/SCANNING in non-secured connection or AUTHENTICATING in
secured connection. The latter prevented the scan. Update the
wpa_supplicant state in mesh to be COMPLETED upon initialization. This
is similar to the P2P GO case.

Signed-off-by: Maital Hahn <maitalm@ti.com>

tests: wpa_supplicant AP mode - WPS operations with fragmentation

Signed-off-by: Jouni Malinen <j@w1.fi>

wpa_supplicant: Configurable EAP fragment size in AP mode

In wpa_supplicant AP mode, allow configuration of the EAP fragment size
using the fragment_size member of network block (wpa_ssid), similar to
the fragment_size in hostapd configuration. bss->fragment_size default
value of 0 is treated specially in some EAP code paths (such as MTU
initialization in eap_pwd_init). In order to preserve the existing
behavior, bss->fragment_size will only be set if the network block
specified a value different from the default which is
DEFAULT_FRAGMENT_SIZE(1398) bytes.

Signed-off-by: Lior David <qca_liord@qca.qualcomm.com>

Assign vendor specific elements for early HE testing

These elements can be used for pre-standard publication testing of HE
before P802.11ax draft assigns the element ID. The payload of these
vendor specific elements is defined by the latest P802.11ax draft.
Please note that the draft is still work in progress and the element
payload is subject to change.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>

PAE: Use big endian version in current_peer_id.mn to be more consistent

This gets rid of sparse warnings related to mismatching annotation and
byte swapping.

Signed-off-by: Jouni Malinen <j@w1.fi>

PAE: Use sci->port more consistently

This is now annotated as be16, so use it as such in all cases instead of
first storing host byte order value and then swapping that to big endian
in other instances of the same structure. This gets rid of number of
sparse warnings.

Signed-off-by: Jouni Malinen <j@w1.fi>

wired: Silence sparse warning on redefinition of IFNAMSIZ

driver_wired.c pulls in utils/common.h before net/if.h as a workaround
for NetBSD build. This results in IFNAMSIZ getting redefined and sparse
warning about this. Silence that warning by undefining the IFNAMSIZ
definition from common.h to allow the one from net/if.h being used. In
addition, remove duplicated inclusion of net/if.h.

Signed-off-by: Jouni Malinen <j@w1.fi>

Move extern declarations for ext_password backends into a header file

This gets rid of a sparse warning and also allows the compatibility of
the declarations to be verified (a missing const declaration is fixed
here as well).

Signed-off-by: Jouni Malinen <j@w1.fi>

Move extern declarations for autoscan modules into a header file

This gets rid of number of sparse warnings and also allows the
compatibility of the declarations to be verified.

Signed-off-by: Jouni Malinen <j@w1.fi>

Move extern declarations for bgscan modules into a header file

This gets rid of number of sparse warnings and also allows the
compatibility of the declarations to be verified.

Signed-off-by: Jouni Malinen <j@w1.fi>

Move extern declarations for driver ops into a header file

This gets rid of number of sparse warnings and also allows the
compatibility of the declarations to be verified (number of missing
const declarations are fixed here as well).

Signed-off-by: Jouni Malinen <j@w1.fi>

nl80211: Mark wpa_driver_nl80211_get_macaddr() static

This function is not used outside this file.

Signed-off-by: Jouni Malinen <j@w1.fi>