Merge "wificond: Remove all libwifi-hal linkage"

wificond: Remove all libwifi-hal linkage

Also, remove the wpa_supplicant binder tests since these are no longer
useful.

Bug: 34457880
Bug: 36100720
Test: Compiles and device boots up.
Change-Id: I3b9ed2c33fce5e8b78d773472130e1010488b92e

Print log when Pno scan started

In addition, We might need to keep using 'pno scan' instead of
'scheduled scan' in this class. This also fixes that.

Bug: None
Test: compile
Change-Id: I989d794583bb5474348c9679e5d444550710bf61

Merge "Add RSSI threshold for each match set respectively."

Temporarily disable wificond integration test

Bug: 35923520
Test: Run this script

Change-Id: I43c6bc73af5d5547b38947b028a817a7987cb163

Merge "wificond: Use Vendor HAL for mode change"

Explicitly returns false on IsValid() failure

Bug: None
Test: compile, unit tests
Change-Id: Ib2e74101e1b6883a5ac7adfa3b53c75d3ccfb877

Add RSSI threshold for each match set respectively.

Some of the devices reject scheduled scan request which
specifies a global RSSI threshold.
This patch uses RSSI threshold for each match set instead.

Bug: 35958492
Test: compile, unit tests, manual test

Change-Id: I0d45a3361c9bf4dbaa9b85ccf655ddfaa5c09899

Merge "Do not return failure if scan results is empty"

Do not return failure if scan results is empty

It may happen if there is no AP in the environment.
In this case we only need to log instead of returning
error.

Bug: None
Test: compile, unit tests
Change-Id: I4187cc421db11c6fcc7e27e263865706cfb61abc

Do not log if NL80211_FREQUENCY_ATTR_FREQ is missing

Bug: None
Test: compile
Change-Id: I2648883013ceb6daa37f13303bc704478c9e9222

wificond: Use Vendor HAL for mode change

Also,
Removed the usage of DriverTool & HalTool from wificond.

Bug: 35765841
Test: Will send for integration tests.
Change-Id: Ie029816bec5b168e34b3b18892b9da82285c66c0

Merge "Get NL80211_BSS_BEACON_TSF for wificond"

Cleanup AP logic after Hostapd is down

Bug: 35445677
Test: compile, unit tests, manual test

Change-Id: Ic7e6c593ded19955da7a85d60730bf8e9afd679a

Get NL80211_BSS_BEACON_TSF for wificond

Bug: 35872043
Change-Id: I68d9dcd2c6cf44efdf4dd843c1ef051e58f142c6
Test: compile, unit tests, manual test

Merge "Fix wificiond scanner integration test"

Merge "Revert "Temrorily disable mac address randomization""

Fix wificiond scanner integration test

This fixes the misuse of unique_ptr.

Bug: None
Change-Id: Ia10ff2e081256b3233b16436b5e3c99db2a5c4dc
Test: compile, unit tests, integration test

Merge "Do not request MAC address randomization when associated"

Do not request MAC address randomization when associated

This CL includes the following changes:
1. Enables netlink manager to monitor NL80211_CMD_DISCONNECT and
NL80211_CMD_DISASSOCIATE events
2. Makes ClientInterfaceImpl listen to these additional MLME events.
3. Makes clientInterfaceImpl keep tracking of the association state.
4. Exposes the association state to ScannerImpl, and makes ScannerImpl
only request MAC address randomization when interface is not associated.

Bug: 35803146
Test: compile, unit tests, manual test

Change-Id: I7a5eeaf6329e1142a0012f27e0f72e6514ba7f94

Revert "Temrorily disable mac address randomization"

This reverts commit 92ab3a7162786698c4309f13c497d91a0836f606.

Change-Id: I75346143aa8935857aad34695ce9bd0d9854199a

Temrorily disable mac address randomization

Bug: 35800644
Test: compile, unit tests

Change-Id: I2b44a4b2de64dea72ec09b0fa756bbaee88851de

Get up-to-date channel info from kernel

Bug: 35707768
Test: compile, unit tests, integration test

Change-Id: Ifb0beb9321d632ec76df3d368c10e4531743e4c3

Change some wificond aidl interfaces to oneway

Some interfaces does not require synchrounous behavior.
We can mark them as oneway.

Bug: 35726133
Change-Id: I7cc9ad5c27b812bc0105c8c52b1e8e25db9721e6
Test: compile, manual test

Do not rely on the destructor of shared object ScannerImpl

We should control the life time of these shared object instead
of relying on java garbage collection.
This will ensure the correct order to avoid race conditions.

Bug: 35664767
Change-Id: I415da1e70c93ceeb79b78c4ce9e833e86313c505
Test: compile, unit tests, manual tests

More logging for wificond scanner

Bug: 35664772
Change-Id: I6b3bb4fc01e3c06dea246e3cd55c23f27d3a2195
Test: compile, manual test

Merge "Wildcard ssid should be zero-length"

Wildcard ssid should be zero-length

Bug: 35441881
Change-Id: I6a1a8683a57a0478330b9b05121cf99481013e3c
Test: compile, unit tests, manual tests

Monitor regulatory domain change

This adds the function of regulatory domain change monitoring
on wificond.

This also allows wificond to print the supported bands/channels
upon regulatory domain change.

This also adds the corresponding unit tests.

Bug: 35150708
Test: compile, unit tests, manual tests
Change-Id: Idbcf9ebf25f4e7be3b371ec3531b6b52303476e8

Merge "Skip scan result with invalid ssid"

Skip scan result with invalid ssid

Bug: 35349025
Change-Id: I05c6d2f524c1b5e8ea5e3fd80443c7e7ff63ed44
Test: compile, unit tests, integration tests

Better logging for GetWiphyInfo()

NL80211_ATTR_WIPHY_BANDS contains a list of anonymous attributes.
Each anonymous attribute represents a band.
NL80211_BAND_ATTR_FREQ is expected to be included in every anonymous
attribute to specifying supported channels for this band.

However, sometimes one of the anonymous attribute provided by kernel is
empty, where NL80211_BAND_ATTR_FREQ is not included. This always happens
on some type of devices. We should not log an error for this.

Bug: 35220903
Test: compile, unit tests, manual test

Change-Id: I0ef953266744713af627a17e7d10f04b937e0648

Remove StartFullScan()

Bug: 35103495
Change-Id: I2004de17898b5c8f59eea88679fe60e6cb57dc87
Test: compile, unit tests

Remove is_full_scan_ from SingleScanSettings

Bug: 35103495
Change-Id: Id4cef15484790cb3ad684cc683903b31996d592b
Test: compile, unit tests

Enforce wificond scanner capability checks

Wiphy reports its capabilities for handling scan requests.
This enables wificond scanner to trim incoming scan request
if request exceeds the wiphy capabilities.
E.g. Framework requests more scanning ssids than the
number wiphy can handle.

This also fixes a few code formatting issues.

Bug: 34685914
Test: compile, unit tests, manual tests

Change-Id: I96d76df13f287f8bf873d9f7818e75604cab3149

Merge "Avoid unnecessary copies of scan result"

Maintain scanning status correctly for wificond scanner

This enable wificond scanner to track one-shot scan and pno
scan status by maintaning |scan_started| and |pno_scan_started|.

This also allows wificond scanner to subscribe scan events from
kernel all the time even if there is no external subscriber.
This ensures wificond scanner keeps tracks of all scan events so
|scan_started| and |pno_scan_started| can be set correctly.

Bug: 33398008
Test: compile, unit tests, manual tests
Change-Id: Id58a77a975e1199f25e14d6f81a39018513aefeb

Avoid unnecessary copies of scan result

This also fixes some typos.

Bug: None
Test: compile, unit tests, manaul tests
Change-Id: Idd3311bc4c9a9e1483e628d2a3856a876aa9aeaf

Merge "Plumb through scheduled scan failures"

Merge changes I1e9eb842,I72046b93

* changes:
  Plumb through scheduled scan results
  Plumb through scan failure events

Merge "Support PNO scan for wificond scanner"

Handle leading number before an object in parcel

Java writeTypedList() implementation uses a leading number
with either 0 or 1 before an object in parcel.
0 means this object is null.
1 means this object is not null.
C++ code should handle it accordingly.

Bug: 34359499
Change-Id: I2e107ffa00573d4ffa31041a14dbec581e09a23a
Test: compile, unit tests, manual tests.

Plumb through scheduled scan failures

Bug: 34611323
Test: compile, unit tests, manual tests

Change-Id: Ib3cc3ae24043c72528f19d27853196c5605b4292

Plumb through scheduled scan results

Bug: 33398008
Test: compile, unit tests, manual test

Change-Id: I1e9eb84294cf99cda377c5c17edda97baa465895

Plumb through scan failure events

Bug: 34611323
Change-Id: I72046b93c1c8270d96d2ead595220cd1ffc6f28a
Test: compile, unit tests

Support PNO scan for wificond scanner

This also fixes the aidl file naming.
Functions should start with lower case alphabet.

Bug: 33398008
Test: compile

Change-Id: Icb401d398c819ce1f54fb32da116e21fb835ec2b

Add a hidden flag to PnoNetwork

We need to specify are the requested networks are hidden when
requesting a PNO scan.

Bug: 33398008
Change-Id: Iab773c59cb4d4ee95bd038eb0445606c8cfdd09e
Test: compile, unit tests

Add support of randomized mac address to wificond ScanUtils

Bug: 33218723
Change-Id: I81e3507f4cc46894b8ae35f0df0814006ddc1ead
Test: compile

Add scan result notification interfaces to wificond scanner

Bug: 33398008
Test: compile

Change-Id: I10eb85b67be4b5b1829c006f92e0cc4d37a9be7d

Add scan() to wificond scanner

Bug: 33398008
Change-Id: Ib2009547ccc8fc368bd92e88a216b61cefe9a8ad
Test: compile

Add getScanResults() to wificond scanner

Bug: 33398008
Change-Id: I40c456f9b2c4ff4ba971cb09f2b5e3904519a250
Test: compile, manual tests

Merge "Add extra comment for length of nested parcelable objects list"

Merge "Remove extra writing of array length in PnoNetwork"

Merge "Remove extra writing of array length in ScanResult"

Merge "Remove extra writing of array length in HiddenNetwork"

Remove extra writing of array length in PnoNetwork

Bug: 34359597
Change-Id: Ibceb624de48000852ee6dc406ec1a9bdbae70ee9
Test: compile, unit test

Add extra comment for length of nested parcelable objects list

Bug: 34359499
Change-Id: Ic7f2da47e14d78e61a3211bc3f54cb73c081ef49
Test: compile

Remove extra writing of array length in ScanResult

Bug: 34359597
Change-Id: I91d0043d95b5094e66bdac9732eef86d03273956
Test: compile, unit tests

Remove extra writing of array length in HiddenNetwork

Bug: 34359597
Test: compile, unit tests
Change-Id: If543f3bb3f8ec4e5a9a5bcbe9f7ff524998d215d

Add unit tests for wificond ScanUtils

Bug: 30915875
Test: compile, unit tests

Change-Id: Ic8f913663110f64dfb0387502e60afdfb130b0b5

Add netlink helper function to request a full scan

Bug: 33251936
Test: compile, unit tests, and manual test
Change-Id: I724adc9cd3a3a439f3d4ccbfb389e6752b6dc2d6

Add interfaces for getting available channels to WifiScannerImpl

This adds the following three aidl interfaces to wificond ScannerImpl:

int[] getAvailable2gChannels();
int[] getAvailable5gNonDFSChannels();
int[] getAvailableDFSChannels();

This also adds the corresponding integration tests.

Bug: 33297764
Test: compile, unit tests, integration tests

Change-Id: Idb14740769aa2ab368f311ae099b53005382a007

Merge changes I16fab137,I528112cf

* changes:
  Add parcelable class PnoSettings
  Add parcelable class PnoNetwork

Add parcelable class PnoSettings

Bug: 33011588
Test: compile, unit tests

Change-Id: I16fab137cd6a9d60bea96955997a43b0674ed709

Add parcelable class PnoNetwork

Bug: 33011588
Change-Id: I528112cf4483313511380fd13daf0e7ac0237d19
Test: compile, unit tests

Add unit tests for NetlinkUtils::GetWiphyInfo()

Bug: 33279651
Test: compile, unit test

Change-Id: Id67fd10ceb3058a0792aa81fce112d4cd5778ba2

Merge "Fix some typos"
am: a4932a137c

Change-Id: Ifd038dc7719ea8372983ec8a00a3192aa608cb2d

Merge "Fix some typos"

Expose wiphy capabilities to ScannerImpl
am: 2c74cae9b5

Change-Id: I46a389483f8b6ca23c58712a0793c7449146fa1b

Add parcelable class SingleScanSettings
am: 3afd158172

Change-Id: I4a2f9f8a1d4057fd10cd1c1f5967998375c5f8b0

Add parcelable class HiddenNetwork
am: 941064744e

Change-Id: Ie3b8ca5d5d7c71ebc0df37b77fcfe1cce8b62692

Add parcelable class ChannelSettings
am: cbdd2d05eb

Change-Id: I5d43185b985779d4e88c6e9ba2d76581a122cdd5

Create parcelable_utils.h for helper macros
am: fa1a494f56

Change-Id: I593cbdd1033f0be4966bbd79deab7682f5ce7e72

Expose wiphy capabilities to ScannerImpl

This adds a struct WiphyFeatures to save wiphy feature flags
from netlink wiphy message. Currently we only add fields for
randomized mac address support.

This also allows ClientInterfaceImpl to expose channel info, scan
capabilities and wiphy features to ScannerImpl object.

Bug: 33216906
Test: compile, unit tests, manual test

Change-Id: Iac9355e45a3cf2011120fc6a91ccfb2dee434951

Add parcelable class SingleScanSettings

This adds parcelable class SingleScanSettings on wificond side.
Class SingleScanSettings is going to be used for Java framework
to send single scan requests.

This also adds the corresponding unit tests.

Bug: 33011588
Change-Id: I071edfca5680ef39b8fa1998ecf14f33ccafe207
Test: compile, unit test

Add parcelable class HiddenNetwork

This adds parcelable class HiddenNetwork on wificond side.
Class HiddenNetwork is going to be used for Java framework
to send scan requests. We only add field SSID for now.

This also adds the corresponding unit tests.

Bug: 33011588
Test: compile, unit test

Change-Id: Ia55a409bb815e40264333da7fa71ca32d450af61

Add parcelable class ChannelSettings

This adds parcelable class ChannelSettings on wificond side.
Class ChannelSettings is going to be used for Java framework
to send scan requests. We only add field frequency for single
scan implementaion. We will add more fields to support batched
scan in later CLs.

This also adds the corresponding unit tests.

Bug: 33011588
Test: compile, unit tests

Change-Id: I6ddb6ed2c7958256ea6d4492fc763d05fa8ea89c

Create parcelable_utils.h for helper macros

This creates a new file parceable_utils.h for helper macros used in
C++ binder parcelable classes.

Currently this only includes macro RETURN_IF_FAILED.

Bug: None
Change-Id: I02b1e7feecb9c3879961f1bf6ac50458332c1a91
Test: compile, unit tests

Add WifiScannerImpl object stub implementation
am: c2b0dce5c3

Change-Id: I015538b9c792c0c00b4f0fd193b5a9801fafaed2

Fix some typos

Bug: None
Change-Id: If062034cdfd023a13544c1f9d02c8e4d1ee200cd
Test: Compile

Add WifiScannerImpl object stub implementation

This includes:
1. Add getWifiScannerImpl() to IClientInterface. This call will
return an IWifiScannerImpl object.

2. Remove scanning result listening functionalities out of the
ClientInterface class. These functionalities will be added back
to ScannerImpl class later.

Bug: 31115360
Test: compile, unit tests, integration tests

Change-Id: Ie4ffcbe0d63b2db61e66093df2dda26493e2cd77

wificond: return association frequency upon signal poll
am: 903b9d029c

Change-Id: Idc7b44f8392c404ca4fc72fd2685d85281500c0c

wificond: return association frequency upon signal poll

This allows wificond to return a cached frequency upon signal poll.
This frequency is updated upon MLME events.

Bug: 31595463
Test: compile, manual test

Change-Id: Iae82a8143ee87c44bade1c376a55025fae61f490

Make wificond scan result parcelable
am: 4d882abea2

Change-Id: I8dbacd014ba5a596f074f5a649ea92f3e194125f

Make wificond scan result parcelable

This includes the following changes:
1. Make ScanResult a parcelable class.
2. Move ScanResult to namespace com.android.server.wifi.wificond
3. Rename ScanResult to NativeScanResult to avoid naming confliction.
4. Add unit test for NativeScanResult class.

Test: compile, unittests, manual test
Bug: 30746278
Change-Id: Ic1225ff52149fcd00855f9c7aa78c48485b0a08b

Use AP bssid for NL80211_CMD_GET_STATION command
am: 4aab829f78

Change-Id: I61913f315166a4031535bedd6479774915df16e3

Monitor mlme event for wificond
am: 042736e71a

Change-Id: I287bd9bf0bed87379fbbb579b133242f3621adb3

Use AP bssid for NL80211_CMD_GET_STATION command

Bug: 32095105
Change-Id: Ic048e3f96a325599001100473948c0e5a3736b2d
Test: compile, runttests.sh

Monitor mlme event for wificond

1. This allows NetlinkManager to monitor MLME events.
2. This adds internal API for tracking MLME events.
3. This creates classes representing different MLME events.
4. ClientInterfaceImpl will update frequency and bssid upon
   ASSOCIATE/CONNECT/ROAM events.

Bug: 31961586
Test: compile, manual tests

Change-Id: I1bd62dfa85596fd262709038fb73fc6e0ff9c5f3

Add RTTController and corresponding aidl interface to wificond
am: eb59d4dd54

Change-Id: I299e49d7159fb3a61baa0485666626626ee9c030

Add RTTController and corresponding aidl interface to wificond

This add RttController interface as a lazy initialization object.
User can get an instance of RttController while register a RttClient
through wificond.

Currently HAL RttController has an optional parameter for choosing
on which network interface will RTT run. However, in order to
hide interface concept from RTT framework as well as keep simplicity,
we don't expose this interface choice via aidl to framework for now.

Bug: 31756849
Test: compile

Change-Id: I2a13374254fd8c6bd7911e5cfbe3297cde25e9e7

Merge "Add internal API for handling scheduled scan results"
am: 57e0bf7dd5

Change-Id: I6312a3669bd8442739fd096f9f46eb3da1a2445f

Merge "Add internal API for handling scheduled scan results"

Merge "Stop system_server for wificond integration test DevMode"
am: c64f5270cd

Change-Id: Id70109a2563450e5e5f7c558fe7c5e164636cd28

Add internal API for handling scheduled scan results

This allows an interface object to subscribe/unsubscribe
schedulded scan results.

Bug: 31839487
Test: compile, manual tests, unit tests

Change-Id: I023fe9ca1546148bdab2ef1458934efbcdee4e76

Merge "Stop system_server for wificond integration test DevMode"

Stop system_server for wificond integration test DevMode

For integration tests, now we use "adb shell stop" to get rid of
framework control, instead of  starting wificond service with a
new name.

Bug: 32097569
Test: comipile, integration test
Change-Id: I725e33e05052fd813f1f7f8c92b7ed1d75c8e187

Fix misleading wificond NL80211 logging messages am: e779ee36c4 am: 02f9c4a17a
am: b83eecfb65

Change-Id: I0b57c709dd7438ec78ad333249b22a23d10b272b

Fix misleading wificond NL80211 logging messages am: e779ee36c4
am: 02f9c4a17a

Change-Id: I5db8a1e259ef37de226d71603c06850afb434233

Fix misleading wificond NL80211 logging messages
am: e779ee36c4

Change-Id: I91874b732813792c2db7dcffb91b09b34b9c2292