Commit Graph

7337 Commits

Author SHA1 Message Date
Chalard Jean
57aa680f18 Merge "Simplification of code to prevent a Log.wtf in expected cases." am: 3ef0c36587 am: fbc68b975a am: 8d69c6dca3
am: 1c44f05067

Change-Id: I80bebcd04c277f6e4b0665fe1253b2309e3bc535
2019-06-17 04:42:04 -07:00
Chalard Jean
1c44f05067 Merge "Simplification of code to prevent a Log.wtf in expected cases." am: 3ef0c36587 am: fbc68b975a
am: 8d69c6dca3

Change-Id: I846943dc91ef3cb6dc7516ba38dc93fb8a80bb88
2019-06-17 04:34:32 -07:00
Chalard Jean
8d69c6dca3 Merge "Simplification of code to prevent a Log.wtf in expected cases." am: 3ef0c36587
am: fbc68b975a

Change-Id: I5f5fa22468c8db09e4eeb3d446220b4ebc396d98
2019-06-17 04:24:08 -07:00
Chalard Jean
3ef0c36587 Merge "Simplification of code to prevent a Log.wtf in expected cases." 2019-06-17 10:37:50 +00:00
Chalard Jean
1e3b152eca Simplification of code to prevent a Log.wtf in expected cases.
This mostly serves to unindent code to make it locally more
readable. It is a functional no-op.

Bug: 135043192
Test: atest FrameworksNetTests
Change-Id: Iad0e9a28670e96a3c953518a0d0ccd77e2f2fa80
2019-06-17 18:01:35 +09:00
Xiao Ma
9e42768f01 Merge "Merge "Restore the Private DNS settings to default mode upon network resets." into qt-dev am: 0cb53125b1 am: bfbb0f7ef9" into qt-r1-dev-plus-aosp
am: b78839e095

Change-Id: Id29b439f342ee0ef090376135f479e23ee2c0ba5
2019-06-16 21:33:06 -07:00
Xiao Ma
b187ae7c7a Merge "Restore the Private DNS settings to default mode upon network resets." into qt-dev am: 0cb53125b1 am: adbde27464
am: 0bac5c866c

Change-Id: Ie4936d48f4a3638f69e909dfc5a61ac848d5ac12
2019-06-16 21:31:11 -07:00
Android Build Merger (Role)
b78839e095 Merge "Merge "Restore the Private DNS settings to default mode upon network resets." into qt-dev am: 0cb53125b1 am: bfbb0f7ef9" into qt-r1-dev-plus-aosp 2019-06-17 04:23:02 +00:00
Xiao Ma
20c4d224e9 Merge "Restore the Private DNS settings to default mode upon network resets." into qt-dev am: 0cb53125b1
am: bfbb0f7ef9

Change-Id: Ib4aa84f745ae26865a94325c8ba8bf8e5d6f9189
2019-06-16 21:22:17 -07:00
Xiao Ma
0bac5c866c Merge "Restore the Private DNS settings to default mode upon network resets." into qt-dev am: 0cb53125b1
am: adbde27464

Change-Id: Id2fb47805befac50c0428eca30ffe93590686c01
2019-06-16 21:21:21 -07:00
Xiao Ma
adbde27464 Merge "Restore the Private DNS settings to default mode upon network resets." into qt-dev
am: 0cb53125b1

Change-Id: I42959ec5d1194851294e007e6e3d872678a85b6d
2019-06-16 21:15:04 -07:00
TreeHugger Robot
0cb53125b1 Merge "Restore the Private DNS settings to default mode upon network resets." into qt-dev 2019-06-17 01:08:00 +00:00
Heemin Seog
d6bb517577 Merge "Change factoryReset to succeed for Settings" am: 67322170b5 am: 9d59a75463 am: 86ee3e922d
am: 6f44822682

Change-Id: Ib8361625c9d0ef00f6cbbb6e35b5167ce8d7dcd6
2019-06-14 00:39:39 -07:00
Heemin Seog
6f44822682 Merge "Change factoryReset to succeed for Settings" am: 67322170b5 am: 9d59a75463
am: 86ee3e922d

Change-Id: I5b826a4744cd4dfc8848ab6cd8313307d537d68b
2019-06-14 00:31:38 -07:00
Heemin Seog
86ee3e922d Merge "Change factoryReset to succeed for Settings" am: 67322170b5
am: 9d59a75463

Change-Id: I9b7ce285fc205255dd9e2e0fa71afd0f8392d243
2019-06-14 00:25:17 -07:00
Treehugger Robot
67322170b5 Merge "Change factoryReset to succeed for Settings" 2019-06-14 07:02:17 +00:00
Lorenzo Colitti
6f2385bbca Merge "Change factoryReset to succeed for Settings" into qt-dev am: a86fb41d68 am: 5015d9fa86
am: 84bd2bc63e

Change-Id: I6537cc1b37184c0db27ff9e93a3b55f38292b2be
2019-06-13 22:46:03 -07:00
Lorenzo Colitti
e0266ba227 Merge "Change factoryReset to succeed for Settings" into qt-dev am: a86fb41d68 am: 2d2d35427c
am: b1e757ebb8

Change-Id: I75b0f141421ff0eec3e9d62daf953600a8173186
2019-06-13 22:34:04 -07:00
Lorenzo Colitti
84bd2bc63e Merge "Change factoryReset to succeed for Settings" into qt-dev am: a86fb41d68
am: 5015d9fa86

Change-Id: Icfc13c71e2d768a1080aeaa9130896621fe986ce
2019-06-13 22:13:35 -07:00
Lorenzo Colitti
b1e757ebb8 Merge "Change factoryReset to succeed for Settings" into qt-dev am: a86fb41d68
am: 2d2d35427c

Change-Id: Ica379a97191bb4c7f7b93b4b3396df9e102d30e0
2019-06-13 22:09:00 -07:00
Lorenzo Colitti
5015d9fa86 Merge "Change factoryReset to succeed for Settings" into qt-dev
am: a86fb41d68

Change-Id: Iea895d4bbbae2fd6ddaf0c7627c8066c0a7e6033
2019-06-13 21:52:34 -07:00
Lorenzo Colitti
a86fb41d68 Merge "Change factoryReset to succeed for Settings" into qt-dev 2019-06-14 04:13:27 +00:00
Heemin Seog
d5085f3093 Change factoryReset to succeed for Settings
Wrap a part of factoryReset with Binder.withCleanCallingIdentity() so
that it doesn't crash thinking that a different uid connected to the
network stack.

Test: build, manual
Change-Id: Iea246a4c1939a4e7e35434137051835ece81d92f
2019-06-13 09:30:19 -07:00
Heemin Seog
47e9a55358 Change factoryReset to succeed for Settings
Wrap a part of factoryReset with Binder.withCleanCallingIdentity() so
that it doesn't crash thinking that a different uid connected to the
network stack.

Bug: 135029349
Test: build, manual
Change-Id: Iea246a4c1939a4e7e35434137051835ece81d92f
2019-06-13 09:19:48 -07:00
Chiachang Wang
d8daf1277d Merge "Suppress the wtf log for notifications that are expected" am: 6b6e383d2b am: 4e3761e1d9 am: 632cf0f090
am: 26fb026e63

Change-Id: I91b92249dc7905aadbc59df50c3bc6da30a8590e
2019-06-13 02:50:16 -07:00
Chiachang Wang
26fb026e63 Merge "Suppress the wtf log for notifications that are expected" am: 6b6e383d2b am: 4e3761e1d9
am: 632cf0f090

Change-Id: I228813f2a79025b51c206cec920e1762b680212f
2019-06-13 02:46:01 -07:00
Chiachang Wang
632cf0f090 Merge "Suppress the wtf log for notifications that are expected" am: 6b6e383d2b
am: 4e3761e1d9

Change-Id: Ia1c2a765b0fb0cc8d440c02533bdc15774a5a3ef
2019-06-13 02:29:06 -07:00
Chiachang Wang
6b6e383d2b Merge "Suppress the wtf log for notifications that are expected" 2019-06-13 09:15:09 +00:00
Chiachang Wang
a6cd3b1b89 Suppress the wtf log for notifications that are expected
PARTIAL and NO_INTERNET may happen in the real world for
those transport types that provide internet. These two
notification types should be reasonable notificaitons, not a
terrible failure as the log. For Q, it may be too risky to
display more notifications with other information instead of
SSID. Thus, suppress the wtf log for these two notifications.

Bug: 135043192
Test: atest FrameworksNetTests
Change-Id: I35f3718fa93b403858587d918f0bc596f6c92f3e
2019-06-13 07:55:07 +00:00
Luke Huang
402e2e5d76 [automerger skipped] Merge "Merge "Fix race condition caused by fd reused for DnsResolver" into qt-dev am: 7187dbc4e0 am: f7e2b55a70" into qt-r1-dev-plus-aosp
am: 47ee9d48ba -s ours
am skip reason: change_id I443bb11b15845b079ee4370a7797e692e62fa3c8 with SHA1 766354ccc3 is in history

Change-Id: I45c5e6309a6218448b82b7226aa3190177058bb4
2019-06-12 01:33:38 -07:00
Luke Huang
e5bdb45668 [automerger skipped] Merge "Fix race condition caused by fd reused for DnsResolver" into qt-dev am: 7187dbc4e0 am: 143d037127
am: 6d76a45289 -s ours
am skip reason: change_id I443bb11b15845b079ee4370a7797e692e62fa3c8 with SHA1 766354ccc3 is in history

Change-Id: I176b4ab3e57377ed5e4b257dc1ba6cb44a1bc8b2
2019-06-12 01:32:02 -07:00
Android Build Merger (Role)
47ee9d48ba Merge "Merge "Fix race condition caused by fd reused for DnsResolver" into qt-dev am: 7187dbc4e0 am: f7e2b55a70" into qt-r1-dev-plus-aosp 2019-06-12 08:27:58 +00:00
Luke Huang
6d76a45289 Merge "Fix race condition caused by fd reused for DnsResolver" into qt-dev am: 7187dbc4e0
am: 143d037127

Change-Id: I3ccce970c788944cbdf597e92766b1fa30d2009e
2019-06-12 01:27:05 -07:00
Luke Huang
07de6202d4 Merge "Fix race condition caused by fd reused for DnsResolver" into qt-dev am: 7187dbc4e0
am: f7e2b55a70

Change-Id: I5be2a7839c7384bd566320ee153944738d91c0f8
2019-06-12 01:27:03 -07:00
Luke Huang
143d037127 Merge "Fix race condition caused by fd reused for DnsResolver" into qt-dev
am: 7187dbc4e0

Change-Id: Ib33d8a7955039eb6ec40c949670a52fa2f76660a
2019-06-12 01:22:07 -07:00
Xiao Ma
00ed61897d Restore the Private DNS settings to default mode upon network resets.
Bug: 134918038
Test: atest FrameworksNetTests NetworkStackTests
Test: manual test
      configure private dns settings (e.g. dns.google) -> connect Wi-Fi
      -> click "Reset Wi-Fi, mobile & Bluetooth" -> verify private dns
      settings is automatic -> connect Wi-Fi -> verify Internet
      connection.

Change-Id: I13562af622d7b584d6219f74534219b7868275f9
2019-06-12 16:31:14 +09:00
Mark Chien
8fd2a20670 Merge "Fix NullPointerException in ConnectivityService" am: 6edc85b8e4 am: d0e6636019 am: 889fe88bf0
am: 47c7468b78

Change-Id: I8ac5bbeb053af4e626980725c8216758c797e6ea
2019-06-11 23:53:47 -07:00
Mark Chien
47c7468b78 Merge "Fix NullPointerException in ConnectivityService" am: 6edc85b8e4 am: d0e6636019
am: 889fe88bf0

Change-Id: Ifb398cef8fb51848b4c1cefa2ef1dfd6b6c01732
2019-06-11 23:48:31 -07:00
Mark Chien
889fe88bf0 Merge "Fix NullPointerException in ConnectivityService" am: 6edc85b8e4
am: d0e6636019

Change-Id: If0c65bd11f5b76e35f6033fa79a8938f7e47a8e7
2019-06-11 23:43:37 -07:00
Mark Chien
6edc85b8e4 Merge "Fix NullPointerException in ConnectivityService" 2019-06-12 06:24:41 +00:00
Varun Anand
726806d7a3 [automerger skipped] Merge changes from topic "vpn_data_accnt_revert" am: aa3d512c05 am: 39f9c1c212 -s ours am: ab1ec6527c -s ours
am: a51b10cae0 -s ours
am skip reason: change_id Ibdaad3a4cbf0d8ef1ed53cfab1e454b9b878bae9 with SHA1 237d3e55cd is in history

Change-Id: I476f1bf69f0cceaeadbe6dbdd93b5dfa096e955a
2019-06-11 19:22:30 -07:00
Varun Anand
a51b10cae0 [automerger skipped] Merge changes from topic "vpn_data_accnt_revert" am: aa3d512c05 am: 39f9c1c212 -s ours
am: ab1ec6527c -s ours
am skip reason: change_id Ibdaad3a4cbf0d8ef1ed53cfab1e454b9b878bae9 with SHA1 237d3e55cd is in history

Change-Id: I48f805c08d46e03a837173f59f411506c9748951
2019-06-11 19:11:23 -07:00
Varun Anand
ab1ec6527c [automerger skipped] Merge changes from topic "vpn_data_accnt_revert" am: aa3d512c05
am: 39f9c1c212 -s ours
am skip reason: change_id Ibdaad3a4cbf0d8ef1ed53cfab1e454b9b878bae9 with SHA1 237d3e55cd is in history

Change-Id: I47d22d76e3525efc7771e6b514ebadc9b2c75e22
2019-06-11 19:03:04 -07:00
Varun Anand
aa3d512c05 Merge changes from topic "vpn_data_accnt_revert"
* changes:
  Revert "Take all VPN underlying networks into account when migrating traffic for"
  Revert "NetworkStatsService: Fix getDetailedUidStats to take VPNs into account."
  Revert "Addressing comments for http://ag/7700679."
  Revert "Add one more test for VPN usage stats."
2019-06-12 01:05:01 +00:00
Luke Huang
d4b823433c Fix race condition caused by fd reused for DnsResolver
There might be a gap between fd close and fd event listener unregister.
If fd is reused for another query during that gap, it might cause the
query failed with no response since addOnFileDescriptorEventListener
method failed. To fix this problem, we must ensure that fd event
listener is unregistered before fd closing.

Bug: 134310704
Test: atest DnsResolverTest

Merged-In: I443bb11b15845b079ee4370a7797e692e62fa3c8
(cherry picked from commit 07de4cf82ac09f8b9f37afa9eb1b7a44b43b6fe6)

Change-Id: I7041e67d8c906cbf88050e7d94245f8e15dcdbb4
2019-06-11 17:33:39 +00:00
Luke Huang
766354ccc3 Merge "Fix race condition caused by fd reused for DnsResolver" am: ece3a7471c am: f8a2c15ca1 am: ac5024452d
am: 8e760fe220

Change-Id: I443bb11b15845b079ee4370a7797e692e62fa3c8
2019-06-11 09:08:00 -07:00
Luke Huang
8e760fe220 Merge "Fix race condition caused by fd reused for DnsResolver" am: ece3a7471c am: f8a2c15ca1
am: ac5024452d

Change-Id: I6268edd2b0f98694de7c55f8c6fac844cd5f8635
2019-06-11 08:56:50 -07:00
Luke Huang
ac5024452d Merge "Fix race condition caused by fd reused for DnsResolver" am: ece3a7471c
am: f8a2c15ca1

Change-Id: I93e0d4a8664a16f2814763346ff00e6c874efd02
2019-06-11 08:52:18 -07:00
Luke Huang
ece3a7471c Merge "Fix race condition caused by fd reused for DnsResolver" 2019-06-11 15:32:17 +00:00
Luke Huang
17a68ce4f8 Fix race condition caused by fd reused for DnsResolver
There might be a gap between fd close and fd event listener unregister.
If fd is reused for another query during that gap, it might cause the
query failed with no response since addOnFileDescriptorEventListener
method failed. To fix this problem, we must ensure that fd event
listener is unregistered before fd closing.

Bug: 134310704
Test: atest DnsResolverTest
Change-Id: Ifb6bd34dc54dcf1c61fe8b87785124df4bc0f410
2019-06-11 15:18:28 +08:00