Notify scan aborted events am: c9566b1dc5
am: bba5bcde7b

Change-Id: Ib2497a4acaa6fb1e111a4e4e36ca535289e9e388

Notify scan aborted events
am: c9566b1dc5

Change-Id: I4e9340e637b529edcd38d86f14743c8f928a9db9

Notify scan aborted events

Bug: 31495091
Test: compile, manual tests

Change-Id: I94c50f44449bd1f0c7c445775434a34d5ca7390a

Support fetching packet counters for wificond am: a12253e46c
am: b04e955e16

Change-Id: I315610705b2d62093642a20e00abaebd82cfa1c6

Support fetching packet counters for wificond
am: a12253e46c

Change-Id: Ibd67b45d41da194bbb4b160872fbfb80c246ea7e

Support fetching packet counters for wificond

BUG=30978872
TEST=compile, manual tests

Change-Id: Ifa0dd9ce794a1cdb6da7e357d6873992091e3ea0

Merge "Add 'get interfaces' to wificond Binder interface." am: 4e7a07d580
am: b2de52e555

Change-Id: I1d65e76f60c5b8802a6cc335aa600bc1f0dd00c2

Merge "Add 'get interfaces' to wificond Binder interface."
am: 4e7a07d580

Change-Id: Ifcff8af9880f5f2761777ad7fbda38d3c2ef637a

Merge "Add 'get interfaces' to wificond Binder interface."

Merge "Handle 0 nla_len for DebugLog()" am: 96b5919172
am: 5c20508090

Change-Id: If79e558c01edeb8e12b49173023eeeac4ea8b0fe

Merge "Handle 0 nla_len for DebugLog()"
am: 96b5919172

Change-Id: I6961a4151a934757fb3c2eeec2c76f2760398600

Merge "Handle 0 nla_len for DebugLog()"

Add 'get interfaces' to wificond Binder interface.

This also adds the missing 'override' to enableSupplicant() and
disableSupplicant().

BUG=31430211
TEST=compile, manual tests

Change-Id: I23db45a41d1dcbf5d3294b50e94961eb2de6bf4a

Add a nl80211 helper function to get band info and scan capabilities am: e9a13390cb
am: 5e24556e6e

Change-Id: I0046ec3694afd29c5f33a0b32402c6eb3ff50dfc

Add a nl80211 helper function to get band info and scan capabilities
am: e9a13390cb

Change-Id: I5dcd28a119216a2dd0dcae75764fb5db14176456

Add a nl80211 helper function to get band info and scan capabilities

This also fixes a few lines of logging.

BUG=31458138
BUG=31429695
TEST=compile, manual tests

Change-Id: I309d9ce82eb80e79a97149bd16e1caf5e13d674b

Handle 0 nla_len for DebugLog()

This fixes the infinite loop caused by 0 nla_len.
This also fixes a typo.

BUG=None
TEST=compile, manual tests

Change-Id: I903d5d3f179a96f1105e0d66f26b59d473996dfe

Merge "Broadcast interface changes through binder" am: 4086519433
am: f341c268dd

Change-Id: I1e3e75a996c2efb4130e9bdd18ba5e3fdf3eb156

Merge "Broadcast interface changes through binder"
am: 4086519433

Change-Id: I69e95b893a89926a2f855bbe66a899b7a66f7272

Merge "Broadcast interface changes through binder"

Merge "Retrieve list of attributes from a nested attribute" am: fc3c67a237
am: 7452dcd4c3

Change-Id: Ie52036b44e153a1070d6abce87b026d6dbd994e0

Merge "Retrieve list of attributes from a nested attribute"
am: fc3c67a237

Change-Id: I447559a6faa4a53a54ff9457453fd3779a948155

Merge "Retrieve list of attributes from a nested attribute"

Broadcast interface changes through binder

This broadcasts interface ready/towndown events after we
create/remove an interface.

BUG=31349441
TEST=compile, unit tests
Change-Id: I8d4f20ddb82db1bb53d12f3feeb06d4e761be689

Merge "Make failure to take ownership of firmware path fatal" am: 47d4d3437d
am: 00cb4dcaee

Change-Id: I5a2a206b3768f900c90a54e0e681e3280890b8b4

Merge "Make failure to take ownership of firmware path fatal"
am: 47d4d3437d

Change-Id: Ib69c5a183725614ce7b9e9c17dd7df9938c476a7

Merge "Make failure to take ownership of firmware path fatal"

Add binder interface for registering/unregistering interface changes am: dea7ebe307
am: faf95f77f2

Change-Id: I65b894d1d44743fe5f921e1c5bf1c10f13434b28

Add binder interface for registering/unregistering interface changes
am: dea7ebe307

Change-Id: I1650a2ee1443f9141f5e828a1057917c3b4ee482

Add binder interface for registering/unregistering interface changes

BUG=31349441
TEST=compile, unit tests
Change-Id: I0fa00ecdeec46246875f7de2479dab1470d102e5

Make failure to take ownership of firmware path fatal

Bug: 31225859
Test: Can connect to wifi networks after this change.
Test: unit, integration tests pass
Change-Id: I79a2657d0a6898dc6db581b4c10d98adcb907bb1

Merge "Add capability field to wificond scan result" am: 5811a5ac46
am: 23baf302a5

Change-Id: I048c6f8823345a38a198b4374a450c952b639115

Merge "Add capability field to wificond scan result"
am: 5811a5ac46

Change-Id: Ie7b537ee9a22cc98c8aecbf2857abe804c174f45

Merge "Add capability field to wificond scan result"

Retrieve list of attributes from a nested attribute

This adds helper functions to retrieve a list of attribute
values or nested attributes from a nested attribute.
This helps us get all sub-attributes in one pass.

BUG=30901326
TEST=compile, unit tests, manual tests

Change-Id: I3f9090b65f8e08c2a2643b35ec6db3340e1effd2

Merge "Return on invalid attribute length for DebugLog()" am: 3a6630a841
am: 6f103e177f

Change-Id: I8b230b1594e110801026bcec5d7a658b388b55ff

Merge "Return on invalid attribute length for DebugLog()"
am: 3a6630a841

Change-Id: Ic12580c71f01853bfd9fcc590e5b3e0112e89354

Merge "Return on invalid attribute length for DebugLog()"

Return on invalid attribute length for DebugLog()

This fixes the case where |nla_len| is invalid and it causes
an infinite loop.

BUG=None
TEST=compile, manual tests

Change-Id: Ica24355618b144337a5731d57a57202c031472a5

Merge "Use wrapper functions to simplify message response handling" am: 2a48fa9794
am: 7ba21d504f

Change-Id: I65b70843c73540c3d94f31c7f1226032cf1d2a76

Add capability field to wificond scan result

BUG=31157181
TEST=compile and manual test

Change-Id: I8c7cef6334a49eedd1060da6d417e771ed049916

Merge "Use wrapper functions to simplify message response handling"
am: 2a48fa9794

Change-Id: I4134c1d7d28a28a8151e8d1e6a5ab4ea0564a461

Merge "Use wrapper functions to simplify message response handling"

Fix looper unit tests am: 26e1a4b76b
am: e8973c613e

Change-Id: I6292d897b3fef95ed66874af553003c9db1f3d11

Fix looper unit tests
am: 26e1a4b76b

Change-Id: Id619b67116a42e7729c1eff77df3cbbc71873371

Fix looper unit tests

|LooperBackedEventLoopStopWatchFdTest| sometimes fails because the
read_result has no initial value.

This patch fixes it and also gives initial value to all boolean
variables for readability.

BUG=None
TEST=unittests
Change-Id: I35c179b64fc262ebbe76469546d6c0158d125485

Merge "Do not assert crash if FW_PATH can not be owned" am: 5dcc50802a
am: 0b8e3cb5ce

Change-Id: Ica53fa0392e61ac7d36a0da469a9748ebaa133cf

Merge "Do not assert crash if FW_PATH can not be owned"
am: 5dcc50802a

Change-Id: I27368ca57d5dc2033514f3a5f12da89c330a1fea

Merge "Do not assert crash if FW_PATH can not be owned"

Fix logging of GetInterfaceInfo() and GetWiphyIndex() am: 3b89469905
am: fd02c4e09a

Change-Id: Idc27fcec2bdd144b74e1c20096d76ed2d5c77d74

Fix logging of GetInterfaceInfo() and GetWiphyIndex()
am: 3b89469905

Change-Id: I64d49229aa935a71eff0019577ccfcc95e2185a0

Do not assert crash if FW_PATH can not be owned

Bug: 29220405

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

Use wrapper functions to simplify message response handling

BUG=31111084
TEST=compile, unittests

Change-Id: I220719f94f768cd8dd9f3711347d3c399389fba0

Fix logging of GetInterfaceInfo() and GetWiphyIndex()

CleanUpSystemState() calls GetWiphyIndex() to mark down
any existing interfaces.
It should not be an ERROR if we find no wiphy/interface
at this time.

BUG=None
TEST=compile

Change-Id: I64c1f81625c4964b5ed8c501bd6f761cde883495

Merge branch 'stage-aosp-master' into nyc-dev-plus-aosp

Do not report error if no ssid is found in IE

BUG=None
TEST=compile

Change-Id: I71b441728687faf167d417e50255322143c17f80

Merge "Add wrapper functions of SendPacketAndGetResponses"

Add wrapper functions of SendPacketAndGetResponses

This can help simplify the existing code of handling netlink
responses.

BUG=31111084
TEST=compile

Change-Id: I9a48ddb9af805c59c2733b7c1e2413b3f049edda

Add function StartScheduledScan()

This function starts scans repeatedly.
This is expected to be used to request Pno scans.

BUG=31067130
TEST=compile and manual tests

Change-Id: I6f0e0bc999197e84a8c6130c068204b305e71d2f

Merge "Add function StopScheduledScan()"

Merge "Tear down system state on wificond startup"

Add DebugLog() to NL80211NestedAttr

BUG=None
TEST=compile, manual test

Change-Id: Id2dd18f937f6f9798723555f78c3f3e72b2440a8

Add function StopScheduledScan()

BUG=31067130
TEST=compile and manual tests

Change-Id: I81bccf991314ff9eb478e98d5c5db17d0561587a

Tear down system state on wificond startup

Test: Added an integration test
Bug: 29620309

Change-Id: Ibf378dda9b6ca2dfa29cca3071d967a3e4f6b036

Use mockable supplicant delegate

This allows us to mock interactions with supplicant in unit tests.

Bug: 29620309
Test: unit/integration tests pass

Change-Id: I1526c90ffa9822a4d7dcabe5752676bfc61d6965

Drop privileges and become wifi user during startup

Bug: 29870863
Test: Unit and integration tests pass

Change-Id: I939ad488eb3ad17cb2c166298e720f7d7b368f57

Set AP interfaces down after stopping hostapd

This cleans up after we unceremoniously kill hostapd with SIGKILL.
It also guarantees that we don't continue beaconing after the death
of hostapd.

Test: Unit and integration tests pass
Bug: 31023120

Change-Id: I6a363bfdab81ea24a25d6a86a42ff4ccc99556dd

Fetch the mac address in GetInterfaceNameAndIndex

Rename the |GetInterfaceNameAndIndex| function to
|GetInterfaceInfo| and fetch the mac address assigned to the interface
along with the other info being fetched.

BUG: 31038272
TEST: Modified existing tests
TEST: Manual tests on bullhead to see if the correct mac address is
fetched.

Change-Id: I89c11e2227ef9bdf90dfd681df09d8b8116cd53f

Get rid of NL80211Packet copying

This CL removes all NL80211Packet copying by introducing the use of
unique_ptr on NL80211Packet objects.

The copy constructor is not explicitly deleted because it helps write
readable unit tests.

BUG=30901326
TEST=compile, unit tests, manual tests

Change-Id: I8dc9b42cacc3e929aa720abbd8ff0a3862b33bd3

Merge changes I502fb470,I5bb0d98c

* changes:
  Monitor scan results for current client interface
  Add helper functions to get and parse scan results

Merge "Pass packet by reference to reduce copy operations"

Merge "Add a copy counters for netlink packet debugging"

Monitor scan results for current client interface

This CL includes the following changes:
  1. Add functions which can subscribe and unsubscribe scan results
  notification in ScanUtils, so we don't need to expose a NetlinkManager
  object to ClientInterfaceImpl.

  2. ClientInterfaceImpl subscribes the scan results when it is
  initialized. It unsubscribes the scan results when it is destroyed.

  3. ClientInterfaceImpl requests scan results objects using ScanUtils.
  In later CLs these objects are supposed to be sent to java framework
  through binder.

  4. Add corresponding mock classes and unit tests adjustment for the
  changes above.

This CL won't enable the scan results monitoring unless we uncomment
SubscribeToEvents() in NetlinkManager::Start().

BUG=30896985
TEST=compile, unit tests, manual tests
Manual test:
  1. Uncomment SubscribeToEvents() in NetlinkManager::Start().
  2. Call ScanResult::DebugLog() in ClientInterfaceImpl::
  OnScanResultsReady().
  After 1 and 2 we can see detail scan results in wificond logs.

Change-Id: I502fb4704ffce4dd2bf8de8d0d5d9293b2b43d9a

Add helper functions to get and parse scan results

GetScanResult() is used for request the newest scan results from kernel.
ParseScanResult() is used for converting scan result packet to a scan
result object.
This CL also moves all the scanning helper functions to a new file.

BUG=30896985
TEST=manual tests
TEST=unit tests

Change-Id: I5bb0d98cb62daf9cc02da31a7cdf42f0c9fff16c

Pass packet by reference to reduce copy operations

With this change, along with ag/1333902 the copy counter is
reduced from 14 to 5, for the entire booting process.

BUG=30901326
TEST=unit tests

Change-Id: Ie7f46b6ca1663a00d1703d567b2de35259bfd13a

Merge "Add the ability to start/stop supplicant to wificond"

Add a copy counters for netlink packet debugging

Copy operation of NL80211Packet copies the entire buffer. This
costs a lot resouces. We should avoid it as much as we can.

This CL adds a static variable in NL80211Packet tracking the number
of copy operations, as well as another counter tracking the bytes of
we copied. This help us debug wificond netlink performance issues.

BUG=30901326
TEST=manual tests

Change-Id: I5be04ff977b5ffd7340a98d435a5703b0b79be28

Merge "Add function for triggering scan"

Add function for triggering scan

This adds a function scan() to send a 'trigger scan' message to kernel.

BUG=30643703
TEST=manual tests

Change-Id: I8be2d8a033f38ca66fd29cc4a7135c8526985032

Disable wificond scan result monitoring for now

This disbales the scan result monitoring of wificond.
We will re-enable it when the integration finishes, otherwise
wificond will silently using CPU/Memory to parse scan results.

BUG=None
TEST=unit tests

Change-Id: Iaa50fd4f2f6b7f60dbfcec08f59e21b539c603bc

Merge "Add a helper function for logging scan result"

Merge "Get SSID from information element"

Add a helper function for logging scan result

This function is only used for debugging now.

Example output:
Scan result:
SSID: GoogleGuest
BSSID: 6c:f3:7f:af:fa:b1
FREQUENCY: 5220
SIGNAL: -86dBm
TSF: 23486915

BUG=30746278
TEST=manual tests

Change-Id: Ia504a0e18723dcbffcd3d8da0920d1ea2238af70

Get SSID from information element

Scan result netlink message doesn't provide SSID directly.
It provides raw information elemnt array, which contains SSID.
This helper function can get SSID from that array.

BUG=30896985
TEST=manul tests

Change-Id: I38f43a3528ddddbdc052333e5d0e15875517c89e

Create functions to handle scan result notification

BUG=30737305
TEST=compile, unit tests, integration tests

Change-Id: I763db062ccb59b71ee358723b8285e1c5f2f8aa2

Add the ability to start/stop supplicant to wificond

Introduce two new methods:

enableSupplicant()
disableSupplicant()

to IClientInterface.  These are used to start/stop
wpa_supplicant from controlling a given interface.
Today, they start/stop the wpa_supplicant service, but
in the future we may use them to add/remove the interface
from supplicant's control.

Bug: 30666540
Test: Can still connect to WiFi on bullhead.

Change-Id: I25fe9e7e5f2e3764cd38cf89902dffd957b474ee

Merge "Get interface index from kernel for wificond"

Merge changes I31237daf,Ie4c8eb49

* changes:
  wpa_supplicant_binder_test: Add reconnect tests
  wpa_supplicant_binder_test: Simple connect tests

Get interface index from kernel for wificond

This CL adds the interface index query to interface name query.

With interface index wificond can assign scan request to specific
interface and ignore scan result from uninteresting interfaces.

BUG=30808053
TEST=compile, unit tests, integration tests

Change-Id: I2a6333dcad4e9faadb7793501762602017125865

Merge "Create a ScanResult class for wificond internal use"

Create a ScanResult class for wificond internal use

An object of this class is used for storing the scan result
of a BSS.

More fields will added to this class if it is neccessary.
Future parcelable scan result class is expected to inherit
from this class.

BUG=30746278
TEST=compile

Change-Id: I5d65a6eb51e4ffdd8b92345eef6a51de61aab236

wpa_supplicant_binder_test: Add reconnect tests

Create a base class for all connect tests and add a
test which exercies the new iface methods:
1. IIface.Disconnect
2. IIface.Reconnect

BUG: 30667354
TEST: Ran
/data/nativetest/wpa_supplicant_binder_test/wpa_supplicant_binder_test

Change-Id: I31237dafbcbc276feef7a2814d2246f0069c12e4

wpa_supplicant_binder_test: Simple connect tests

Add a new test file for connection tests. All the tests in this file
parse a json formatted |NetworkParams| structire from command line args.

Changes in the CL:
1. Store any incoming test args in a global |kGlobalTestArgs| variable.
These can be used by the tests as needed.
2. Add a data type |NetworkParams| which is used to deserialize json
formatted command line args to parameters that can be used to connect to
a network.
3. Add a simple connect/disconnect using the above.

BUG: 30667354
TEST: Initiated connection to open/psk networks around.
Open Networks:
/data/nativetest/wpa_supplicant_binder_test/wpa_supplicant_binder_test
'{\"NetworkParams\":{\"ssid\":\"<SSID>\"}}'
PSk Networks:
/data/nativetest/wpa_supplicant_binder_test/wpa_supplicant_binder_test
'{\"NetworkParams\":{\"ssid\":\"<SSID>\",\"psk_passphrase\":\"<PSK>\"
,\"key_mgmt\":2}}'

Change-Id: Ie4c8eb49ea5f10bfd7ba3cde19aac87ab3c85e64

Process all responses to queries for interface name

Sometimes the kernel will tell us about unexpected network devices (e.g.
p2p devices without a real netdev backing them).

Also added a quick facility to log these packets in excruciating detail.
This should only be used for debugging.

Bug: 30773545
Test: angler can now configure client network interfaces in this state.
      added a unittest

Change-Id: I5423acd7dc6177f73e54150e5ef734d15703163d

Merge "Support receiving broadcast for NetlinkManager"

Support receiving broadcast for NetlinkManager

BUG=30737305
TEST=unit tests and integration tests

Change-Id: Ia9fb32d428970560452949b6f980b634a48c84e4

Merge "Add unit tests for NetlinkUtils"

Add unit tests for NetlinkUtils

This creates unit tests for helper class NetlinkUtils.
This also creates mock classes for NetlinkManager and EventLoop.

BUG=30642046
TEST=unit tests and integration tests

Change-Id: Idc3cea812e700af8a1727c380ec92cfc1f364e28

Merge "Fix netlink errno parsing"

Fix netlink errno parsing

Kernel uses negative errno for nl80211 messages.

BUG=None
TEST=compile and manual tests

Change-Id: I50f5fa82ec946f8121c846017023a76e3597589e