summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Aaron Huang <huangaaron@google.com> 2021-05-31 11:07:54 +0000
committer Lorenzo Colitti <lorenzo@google.com> 2021-06-01 07:33:01 +0000
commit40b249c1a6ae1aae7e3b4641bed0335e3dba11da (patch)
treeba3883561a286af37dac3ff5d279c7fb6d97e3fe
parent947270efc3cf6ae619b6e720807eb5fe5b246396 (diff)
Make sure the PAC script content is sent again
...when the PAC service crashes. Bug: 185409199 Test: manual Original-Change: https://android-review.googlesource.com/1711211 Merged-In: Ic76428f42f37f8552a526190764ad32e2874adb0 Change-Id: Ic76428f42f37f8552a526190764ad32e2874adb0
-rw-r--r--services/core/java/com/android/server/connectivity/PacProxyService.java9
1 files changed, 8 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/connectivity/PacProxyService.java b/services/core/java/com/android/server/connectivity/PacProxyService.java
index d23b488f3054..00703390a118 100644
--- a/services/core/java/com/android/server/connectivity/PacProxyService.java
+++ b/services/core/java/com/android/server/connectivity/PacProxyService.java
@@ -345,7 +345,14 @@ public class PacProxyService extends IPacProxyManager.Stub {
if (mProxyService == null) {
Log.e(TAG, "No proxy service");
} else {
- mNetThreadHandler.post(mPacDownloader);
+ // If mCurrentPac is not null, then the PacService might have
+ // crashed and restarted. The download task will not actually
+ // call setCurrentProxyScript, so call setCurrentProxyScript here.
+ if (mCurrentPac != null) {
+ setCurrentProxyScript(mCurrentPac);
+ } else {
+ mNetThreadHandler.post(mPacDownloader);
+ }
}
}
}