diff options
author | Erik Kline <ek@google.com> | 2018-02-21 14:54:04 -0800 |
---|---|---|
committer | Erik Kline <ek@google.com> | 2018-02-21 15:10:01 -0800 |
commit | 36fd22bb9615a313c68c4f74427e7f8f357934b9 (patch) | |
tree | 781ff0cb8272d64b42d954d00bfec95bff89e8a7 /java | |
parent | 22d5a7067cc686f84aaed136b80c9491732af23f (diff) | |
download | ethernet-36fd22bb9615a313c68c4f74427e7f8f357934b9.tar.gz |
Wait for IpClient to shutdownandroid-wear-p-preview-2android-p-preview-3android-p-preview-2android-p-preview-1android-o-mr1-iot-release-1.0.0android-o-mr1-iot-preview-8android-o-mr1-iot-preview-7android-n-iot-release-lg-thinq-wk7o-mr1-iot-preview-8o-mr1-iot-preview-7
This makes sure that we never accidentally have two IpClients vying
for control of the same interface.
Test: as follows
- built
- flashed
- booted
Bug: 62476366
Change-Id: Iab4ca7c2445e70b6f8beaf5dc6921cb0c6cd56ed
Diffstat (limited to 'java')
-rw-r--r-- | java/com/android/server/ethernet/EthernetNetworkFactory.java | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/java/com/android/server/ethernet/EthernetNetworkFactory.java b/java/com/android/server/ethernet/EthernetNetworkFactory.java index d464879..f95500e 100644 --- a/java/com/android/server/ethernet/EthernetNetworkFactory.java +++ b/java/com/android/server/ethernet/EthernetNetworkFactory.java @@ -355,8 +355,10 @@ public class EthernetNetworkFactory extends NetworkFactory { void stop() { if (mIpClient != null) { mIpClient.shutdown(); + mIpClient.awaitShutdown(); mIpClient = null; } + // ConnectivityService will only forget our NetworkAgent if we send it a NetworkInfo object // with a state of DISCONNECTED or SUSPENDED. So we can't simply clear our NetworkInfo here: // that sets the state to IDLE, and ConnectivityService will still think we're connected. |