summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Snap for 4391495 from 27eb79334960933cb82808a57c7c124baeb1390d to oc-mr1-releasestaging/lineage-15.1android-build-team Robot2017-10-121-1/+1
|\ | | | | | | Change-Id: Ie19edb512ffd341a44b4af38e8c5f3597914f75f
| * Merge "Change self recovery log level from wtf to error" into oc-mr1-devTreeHugger Robot2017-10-111-1/+1
| |\
| | * Change self recovery log level from wtf to errorNingyuan Wang2017-10-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reasons: 1. These recovery events are reported to metrics as well. 2. These recovery events are not severe and don't impact user experiecne. 3. We usually can't get useful information from the wtf log snippets. Bug: 66996979 Test: compile, unit tests Change-Id: I713a243927bf935aaf356289be42389f1ba0d5ea
* | | Snap for 4383937 from 0a6b9c1199f91790606acc70c771548f079fa8f0 to oc-mr1-releaseandroid-build-team Robot2017-10-081-2/+7
|\| | | | | | | | | | | Change-Id: Ib79e87b17c92b0edb6168cd1760877ac4b8ffce6
| * | Catch Exception on creating NetworkDetailNingyuan Wang2017-10-061-2/+7
| |/ | | | | | | | | | | Bug: 67482129 Test: compile, unit tests Change-Id: I8d6806f750a8002a0a568bb2b84d1f18ba833594
* | Snap for 4378450 from 87d7badd662aa84369b2b9893e990cb24197ad24 to oc-mr1-releaseandroid-build-team Robot2017-10-0510-111/+857
|\| | | | | | | Change-Id: I364d7180d69955561699e4e525cd47880fccaf95
| * Merge "Do not trigger scan for reconnect() when wifi is connected" into ↵TreeHugger Robot2017-10-052-0/+34
| |\ | | | | | | | | | oc-mr1-dev
| | * Do not trigger scan for reconnect() when wifi is connectedNingyuan Wang2017-10-042-0/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to the document of WifiManager.reconnect(), we only try to connect when wifi is disconnected. This CL adds the handling of CMD_RECONNECT in WifiStateMachine for L2ConnectedState to avoid unnecessary scans and connection attempts. This also adds the handling of CMD_RECONNECT for WpsRunningState, which just logs and ignores this command because in WpsRunningSate we are attempting to connect and when we leave and head to disconnected state, we will trigger a scan anyway. Bug: 65839442 Test: compile, unit tests Test: 1. Write a simple cts test case WifiManagerTest#testWifiReconnect only calling WifiManager.reconnect() 2. Make sure wifi is connected 3. Run cts test using: cts-tradefed run cts -m CtsNetTestCases -t android.net.wifi.cts.WifiManagerTest#testWifiReconnect 4. Grab bugreport 5. Verify the following log appears: "Ignore CMD_RECONNECT request because wifi is already connected" Change-Id: I7aa7842653aa4eaf8d70946dc8e120d3814ffc09
| * | Merge "Blame scan request from WifiManager.reconnect()" into oc-mr1-devTreeHugger Robot2017-10-054-17/+19
| |\|
| | * Blame scan request from WifiManager.reconnect()Ningyuan Wang2017-10-044-17/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 65839442 Test: compile, unit tests Test: 1. Write a simple cts test case WifiManagerTest#testWifiReconnect only calling WifiManager.reconnect() 2. Make sure wifi is disconnected 3. Run cts test using: cts-tradefed run cts -m CtsNetTestCases -t android.net.wifi.cts.WifiManagerTest#testWifiReconnect 4. Grab bugreport 5. Verify the following logs appears: forceConnectivityScan in request of WorkSource{...} Change-Id: I7fa7803e90ba950c9331a55cb1268baef4ae02a8
| * | Merge "DO NOT MERGE Allow the USE_OPEN_WIFI_PACKAGE to access WifiInfo ↵TreeHugger Robot2017-10-054-42/+534
| |\ \ | | |/ | |/| | | | unfiltered." into oc-mr1-dev
| | * DO NOT MERGE Allow the USE_OPEN_WIFI_PACKAGE to access WifiInfo unfiltered.Jeremy Joslin2017-10-044-42/+534
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not remove the SSID/BSSID from the WifiInfo in getConnectionInfo() if the requester is: * The Settings.Global.USE_OPEN_WIFI_PACKAGE * The current user or has been granted INTERACT_ACROSS_USERS_FULL This only applies when the device is connected to an open network. In all other cases the SSID/BSSID will be removed unless the caller has scan result access. Bug: 65639900 Test: ./frameworks/opt/net/wifi/tests/wifitests/runtests.sh Test: Local testing with app. Change-Id: I71eddca75a84945775be476f9ccaf1bb5c668fec
| * | Merge "Allow connecting with no requests if we're already connected." into ↵TreeHugger Robot2017-10-054-46/+216
| |\ \ | | | | | | | | | | | | oc-mr1-dev
| | * | Allow connecting with no requests if we're already connected.Lorenzo Colitti2017-10-044-46/+216
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, we drop all connection start attempts if we do not have any NetworkRequests that we can fulfil. This can happen if the default network has a score that's higher than our score filter. The reason for this is that we don't want to connect a new network if we're not already connected: if we did, ConnectivityService would immediately request that the network be torn down. However, it does make sense to connect if we are already connected, because that means that we ourselves are the highest-scoring network - otherwise, ConnectivityService would have torn us down. So, when deciding whether to process a connection request, allow it if mNetworkAgent is non-null, even if there are no NetworkRequests that we can fulfil. In order to do this, move the check from startConnectToNetwork to the code that processes CMD_START_CONNECT, because it's not safe to access mNetworkAgent except from the handler thread. Moving the check also seems more correct in general, because the return value of hasConnectionRequests() could change between when startConnectToNetwork is called and when the resulting CMD_START_CONNECT is processed. The connect requests are only allowed from the settings/sysui. We don't allow any non-privileged apps to be able to override the user's decision to stay connected to a network with no internet access (which causes the mNetworkAgent.releaseNetworkFor()). Bug: 65262556 Test: Unit tests for START_CONNECT handling for 4 scenarios: a) START_CONNECT with the default network request. b) START_CONNECT with no network request. c) START_CONNECT with no network request, but already connected d) START_CONNECT with no network request, but already connected, but from non-privileged app. Test: Settings can connect to a new network when wifi has a score of 100 Change-Id: Iad582b5109a81f238f335eaac3e7264c5da6e9b1
| * | | Remove hostapd.conf before starting TetheringDaisuke Niwa2017-10-041-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It fails to start Wi-Fi Tethering at first boot after OS version up from Android N to O. It is caused by permission. Owner of hostapd.conf is "system" on Android N. However, it is "wifi" on Android O. hostapd.conf doesn't need to be handed over because it is created again when Tethering is started. Bug: 67351973 Test: manually run tethering Change-Id: Ia7c87ab1f2040434963f6e0670ccd02cdd84bee7
| * | | Merge "Fail scan request if no channels are available" into oc-mr1-devTreeHugger Robot2017-10-042-6/+50
| |\ \ \
| | * | | Fail scan request if no channels are availableNingyuan Wang2017-09-282-6/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fails scan request if no available channels could be scanned for this request. With this fix we can avoid the freezing of WifiScanner in aformentioned case. Bug: 67015387 Test: compile, unit test, integration test Change-Id: I68654ce8d53104484c1559d8b0525585ab50b9e5
* | | | | Snap for 4376088 from 988256f6c383c2c6b24d295c2f2e7ce77ec9955f to oc-mr1-releaseandroid-build-team Robot2017-10-042-3/+53
|\| | | | | | | | | | | | | | | | | | | Change-Id: I69ffda90e6be4e715c156c6b9767f7b95e69ddad
| * | | | Merge "WifiConfigManager: Don't reset the userId on user stop" into oc-mr1-devTreeHugger Robot2017-10-042-3/+53
| |\ \ \ \ | | |_|/ / | |/| | |
| | * | | WifiConfigManager: Don't reset the userId on user stopRoshan Pius2017-09-282-3/+53
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Couple of fixes in user stop handling: a) Only clear the network data belonging to the user that is stopped. b) Don't reset the user Id on stop. It will be done in the user switch handling. Bug: 65939780 Test: Unit tests Change-Id: Ia3ab033d893009cfd8019ee27d52a490a8040438
* | | | | release-request-53526352-7b51-4ab1-a661-632ffc55dd7c-for-git_oc-mr1-release- ↵android-build-team Robot2017-10-0212-77/+784
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4371241 snap-temp-L10900000107789672 Change-Id: I69ee67c044892913b3aa5dc6766f5c683a91733b
| * | | | Merge changes I38c31645,Ib6f61b81 into oc-mr1-devRoshan Pius2017-09-296-67/+304
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | * changes: WifiStateMachine: Handle WifiManager.save() when wifi is off NetworkListStoreData: Set creatorUid for all networks on load
| | * | | WifiStateMachine: Handle WifiManager.save() when wifi is offRoshan Pius2017-09-282-54/+158
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 66909738 Test: Verified that you can save networks even when wifi is off using ag/2930541. Test: Unit tests Change-Id: I38c316456e74ce5ad1ec43750b075f0999c28327
| | * | | NetworkListStoreData: Set creatorUid for all networks on loadRoshan Pius2017-09-264-13/+146
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some network configurations saved during N have the creatorUid set to -1. (Not sure what was the root cause in N which introduced this issue in saved networks) Ensure that we set |creatorUid| for such networks with invalid uid to SYSTEM_UID on loading. The networks with invalid creatorUid can anyway only be modified only by settings/sysui (because they possess NETWORK_SETTINGS permission). Bug: 65623732 Test: Unit tests Change-Id: Ib6f61b81e19877cdf0c42d29c9f5ae3869725517
| * | | | Merge "WifiMetrics: add methods to increment ONA metrics." into oc-mr1-devTreeHugger Robot2017-09-295-8/+438
| |\ \ \ \
| | * | | | WifiMetrics: add methods to increment ONA metrics.Stephen Chen2017-09-285-8/+438
| | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tracks the number of notifications posted for each state of ONA, and the number of user actions made for each notification type. Also tracks the state of the feature setting, the size of the recommender blacklist, recommendation update counts, and connection initiation failure counts. Also adds additional debug logging. Bug: 37501326 Bug: 66996715 Test: frameworks/opt/net/wifi/tests/wifitests/runtests.sh and verify in dumpsys Change-Id: Ia00ae95fbcf923ebae051ba82cd3174cfe11be31
| * | / / Cherry-pick: Remove temporarily disabled networks from PNO listNingyuan Wang2017-09-292-2/+42
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also adds corresponding unit tests. Bug: 66910426 Test: compile, unit tests Merged-In: I58f7061887ebc1dd78d3941218653460f80abb1f Change-Id: I58f7061887ebc1dd78d3941218653460f80abb1f
* | | | release-request-ca7a0f27-e01e-4493-95dc-73b265f88cc9-for-git_oc-mr1-release- ↵android-build-team Robot2017-09-282-3/+71
|\| | | | | | | | | | | | | | | | | | | | | | | 4365318 snap-temp-L33600000106883904 Change-Id: I90ee8d18c349171089514d4f2bf523afddfd5cb4
| * | | Merge "WifiStateMachine: check for null config after dhcp" into oc-mr1-devRebecca Silberstein2017-09-272-3/+71
| |\ \ \ | | |/ / | |/| |
| | * | WifiStateMachine: check for null config after dhcpRebecca Silberstein2017-09-072-3/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a network config is removed while we are in the ObtainingIpState, we will attempt to get the configured network from WifiConfigManager but it will be null. As we move to the Connected state, this throws a NPE. Instead of moving along to the Connected state, disconnect from the network if we find out we have null for the current config. Also add an additional null check where the NPE was thrown further down the line. Bug: 65257934 Test: frameworks/opt/net/wifi/tests/wifitests/runtests.sh Change-Id: Ibbce6491970de16cf5265b03398b50a7cdd50ba2
* | | | release-request-bbc747ae-190b-4d38-8704-432aa6396c16-for-git_oc-mr1-release- ↵android-build-team Robot2017-09-212-13/+35
|\| | | | | | | | | | | | | | | | | | | | | | | 4351869 snap-temp-L93800000104806132 Change-Id: I5560dbb11164ab7ebf1bbcc907271bee927ab3f6
| * | | Fix case for not reverting country code on cellular lossNingyuan Wang2017-09-122-13/+35
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This CL allows us to keep using last known country code on cellular loss and |mRevertCountryCodeOnCellularLoss| is set to false. This also fixes simCardRemoved() to revert country code unconditionally, and cleans up code for airplaneModeEnabled(). Bug: 65602314 Test: compile, unit tests Change-Id: I7b15d9ae197c5e2aacd7788b505e386f970786ed
* | | release-request-b613f8ce-05b5-465e-b783-c1b87f3c1e95-for-git_oc-mr1-release- ↵android-build-team Robot2017-09-133-2/+9
|\| | | | | | | | | | | | | | | | | 4332123 snap-temp-L59300000101925107 Change-Id: I0dcd7a3a5f38da71fca0986aaf98417ae8726067
| * | Fix WifiStateMachineTestNingyuan Wang2017-09-121-1/+1
| | | | | | | | | | | | | | | | | | | | | Bug: 65599523 Bug: 65550505 Test: compile, unit tests Change-Id: Ie9092dae5b07ac14889545c41b2ca212eca97294
| * | capture BR on wificond or HAL crashesNingyuan Wang2017-09-113-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This allows us to save the most recent kernel logs when wificond or hal crashes. This saved log could help us debug underlying driver/kernel issues. Bug: 65550505 Bug: 64934608 Test: compile Test: run frameworks/opt/net/wifi/tests/wifitests/runtests.sh Test: manually kill wificond and take bugreport Find if there is a "Bug dump" section with "reason = 8" Change-Id: Id9311cee34b3fc25400e804bd96795c7f28f537a
* | | release-request-ad401328-e365-4073-9b7c-44f7fc79185b-for-git_oc-mr1-release- ↵android-build-team Robot2017-09-124-33/+137
|\| | | | | | | | | | | | | | | | | 4329935 snap-temp-L86100000101558881 Change-Id: I354eee94a17ecbcd5b344f2455a6a789a96d2c15
| * | Merge "[AWARE] Support multiple NAN data interfaces (NDI)" into oc-mr1-devEtan Cohen2017-09-114-33/+137
| |\ \
| | * | [AWARE] Support multiple NAN data interfaces (NDI)Etan Cohen2017-09-054-33/+137
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support all the NAN data-interfaces which the firmware supports - based on the returned capability. - Create the data-interfaces - Assign data-paths to data-interfaces based on constraints of unique [(src mac, dst mac), security config] tuples. I.e. a single data-interface can only have a single security configuration to a specific destination interface. Bug: 63635857 Test: unit tests + integrated test suite Change-Id: I4a21f3b025e8164673366e22da062d536d588148
* | | | release-request-946c768b-83eb-4b7b-83e7-5737856fa8ec-for-git_oc-mr1-release- ↵android-build-team Robot2017-09-107-200/+531
|\| | | | | | | | | | | | | | | | | | | | | | | 4326576 snap-temp-L80300000101054689 Change-Id: Ic238c9bb3a576e1ffdf5e0f33562950a36852250
| * | | Merge "ONA: Enable new UI and connection flow." into oc-mr1-devStephen Chen2017-09-085-100/+106
| |\ \ \
| | * | | ONA: Enable new UI and connection flow.Stephen Chen2017-09-075-100/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new Open Network Available notification UI introduces a connection flow in which users can connect to a recommended open network directly by interacting with the notification. "Connect" and "See all" action buttons are introduced to initiate the flow, and once a connection attempt is made, the connection status will be reflected on the notification. Bug: 37357441 Test: frameworks/opt/net/wifi/tests/wifitests/runtests.sh Change-Id: I30231c4f71476a7417a3b423848575d5488f51f7
| * | | | Merge "ONA: Add connection related notifications." into oc-mr1-devTreeHugger Robot2017-09-085-154/+479
| |\| | | | | |_|/ | |/| |
| | * | ONA: Add connection related notifications.Stephen Chen2017-09-075-154/+479
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change introduces the new notifications but does not utilize them to change the existing functionality yet. It also refactors the state handling of the notifier to represent the new connection and notification states. When attempting to connect to a network, the notification will update based on connection success and failures. Bug: 37357441 Bug: 65257065 Test: frameworks/opt/net/wifi/tests/wifitests/runtests.sh Change-Id: I3cf188f7f0fac1ea7d1ec882c9473bdbaccccabd
* | | | release-request-8cd2a23c-f767-488e-8139-390d1636483f-for-git_oc-mr1-release- ↵android-build-team Robot2017-09-0813-2124/+150
|\| | | | | | | | | | | | | | | | | | | | | | | 4323561 snap-temp-L80400000100600189 Change-Id: I8b66594d3720a5f2562438da24a05b5fbffa3c41
| * | | Merge "ONA: Implement connection attempt and failure callback." into oc-mr1-devTreeHugger Robot2017-09-076-15/+146
| |\| |
| | * | ONA: Implement connection attempt and failure callback.Stephen Chen2017-09-066-15/+146
| | | | | | | | | | | | | | | | | | | | | | | | Bug: 37357441 Test: frameworks/opt/net/wifi/tests/wifitests/runtests.sh Change-Id: I2d531304598249dcb18bdc45492e48ee90ddff5f
| * | | Merge "WifiConfigStore: Remove legacy modules" into oc-mr1-devRoshan Pius2017-09-079-2109/+4
| |\ \ \
| | * | | WifiConfigStore: Remove legacy modulesRoshan Pius2017-08-319-2109/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These modules were used to migrate store data out of the old store files. Now that O is released, these classes are unused so cleaning up dead code. Bug: 31041781 Test: Unit test passes. Test: Flashed a device and ensured that none of the saved networks are lost. Change-Id: Ib6936cace3c72408ba4be2e33226737228159a48
* | | | | release-request-108343c4-ad88-44f9-aaa2-24d8b8a5c176-for-git_oc-mr1-release- ↵android-build-team Robot2017-09-078-71/+476
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4321077 snap-temp-L97000000100182150 Change-Id: I6389ef4349976122eafc5615b81a3ea022da7c62
| * | | | Handle a condition when the result of getting configured networks returns ↵Aleks Rozman2017-09-071-3/+7
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | null am: 3abc7f0561 am: 3b17189719 am: 1e779938f2 am: fb9027f897 Change-Id: I2198545cb8000378e7350c9d2e580da74af2e3b9