summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Rubin Xu <rubinxu@google.com> 2025-03-24 02:23:41 -0700
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2025-03-24 02:23:41 -0700
commit1135382960db65c5bdaa97e7f61ce57ab5a3b76c (patch)
treed48fa4c3521c85cea7534e9502f4dc32526e4248
parentee18dbb30b7946ab8975dfbeea8489f97815f5d9 (diff)
parent07773b2a8609ae9b960559c1b7d66d64381a2e4b (diff)
Merge "Disable client side binder caching for Pac Service" into main
-rw-r--r--services/core/java/com/android/server/connectivity/PacProxyService.java5
1 files changed, 4 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 2e90a3d86161..c8c1eddd53e7 100644
--- a/services/core/java/com/android/server/connectivity/PacProxyService.java
+++ b/services/core/java/com/android/server/connectivity/PacProxyService.java
@@ -36,6 +36,7 @@ import android.net.Uri;
import android.os.Handler;
import android.os.HandlerThread;
import android.os.IBinder;
+import android.os.IServiceManager;
import android.os.RemoteCallbackList;
import android.os.RemoteException;
import android.os.ServiceManager;
@@ -355,7 +356,9 @@ public class PacProxyService extends IPacProxyManager.Stub {
} catch (RemoteException e1) {
Log.e(TAG, "Remote Exception", e1);
}
- ServiceManager.addService(PAC_SERVICE_NAME, binder);
+ // Do not cache the service, otherwise it will crash com.android.pacprocessor
+ ServiceManager.addService(PAC_SERVICE_NAME, binder, /* allowIsolated */ false,
+ IServiceManager.FLAG_IS_LAZY_SERVICE);
mProxyService = IProxyService.Stub.asInterface(binder);
if (mProxyService == null) {
Log.e(TAG, "No proxy service");