| commit | 8e6c5427dc13d4b67c19ccc83864eca3684ca3c3 | [log] [tgz] |
|---|---|---|
| author | Treehugger Robot <treehugger-gerrit@google.com> | Mon Jan 30 21:39:24 2023 +0000 |
| committer | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | Mon Jan 30 21:39:24 2023 +0000 |
| tree | c35183a0f4c5b58b487cc99e5137645a963373e7 | |
| parent | 2d4c57c0ea0ebe88c1a94eb83fa924b9f35d9b28 [diff] | |
| parent | c2063943e3cbb51ce31292112764bcf2d8bb56d8 [diff] |
Merge "ethernet: prevent provisioning failure loop" am: 210f3b234e am: c2063943e3 Original change: https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/2388903 Change-Id: I9e2e912ad0bfcb3878a223aa640f398eb90ab3cc Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/service-t/src/com/android/server/ethernet/EthernetNetworkFactory.java b/service-t/src/com/android/server/ethernet/EthernetNetworkFactory.java index 51683de..60485f1 100644 --- a/service-t/src/com/android/server/ethernet/EthernetNetworkFactory.java +++ b/service-t/src/com/android/server/ethernet/EthernetNetworkFactory.java
@@ -657,7 +657,17 @@ } void restart() { - if (DBG) Log.d(TAG, "reconnecting Ethernet"); + if (DBG) Log.d(TAG, "restart IpClient"); + + if (mIpClient == null) { + // If restart() is called from a provisioning failure, it is + // possible that link disappeared in the meantime. In that + // case, stop() has already been called and IpClient should not + // get restarted to prevent a provisioning failure loop. + Log.i(TAG, String.format("restart() was called on stopped interface %s", name)); + return; + } + stop(); start(); }