From db8489dc299934c3f598f52ab1534f5388aa7424 Mon Sep 17 00:00:00 2001 From: Heemin Seog Date: Wed, 12 Jun 2019 09:21:44 -0700 Subject: [PATCH] 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 --- .../core/java/com/android/server/ConnectivityService.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/ConnectivityService.java b/services/core/java/com/android/server/ConnectivityService.java index f6a972b090..fcecb71150 100644 --- a/services/core/java/com/android/server/ConnectivityService.java +++ b/services/core/java/com/android/server/ConnectivityService.java @@ -6892,8 +6892,10 @@ public class ConnectivityService extends IConnectivityManager.Stub final int userId = UserHandle.getCallingUserId(); - final IpMemoryStore ipMemoryStore = IpMemoryStore.getMemoryStore(mContext); - ipMemoryStore.factoryReset(); + Binder.withCleanCallingIdentity(() -> { + final IpMemoryStore ipMemoryStore = IpMemoryStore.getMemoryStore(mContext); + ipMemoryStore.factoryReset(); + }); // Turn airplane mode off setAirplaneMode(false);