From c9f24746f2cfa7d3402e2b8f7b048bf8fcc26a75 Mon Sep 17 00:00:00 2001 From: Remi NGUYEN VAN Date: Sun, 10 May 2020 16:11:11 +0900 Subject: [PATCH] Send full LinkProperties to NetworkMonitor Set the parcelSensitiveFields bit when sending LinkProperties to NetworkMonitor, so that the captive portal API URL is not lost. Test: atest ConnectivityServiceIntegrationTest (see followup change) Bug: 156062304 Change-Id: Ifd4e9c02a6b9a2b2b8b254fc4da7bfb9e0a84550 --- .../core/java/com/android/server/ConnectivityService.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/services/core/java/com/android/server/ConnectivityService.java b/services/core/java/com/android/server/ConnectivityService.java index 58e06e00ef..f0a2f59133 100644 --- a/services/core/java/com/android/server/ConnectivityService.java +++ b/services/core/java/com/android/server/ConnectivityService.java @@ -5969,7 +5969,8 @@ public class ConnectivityService extends IConnectivityManager.Stub // Start or stop DNS64 detection and 464xlat according to network state. networkAgent.clatd.update(); notifyIfacesChangedForNetworkStats(); - networkAgent.networkMonitor().notifyLinkPropertiesChanged(newLp); + networkAgent.networkMonitor().notifyLinkPropertiesChanged( + new LinkProperties(newLp, true /* parcelSensitiveFields */)); if (networkAgent.everConnected) { notifyNetworkCallbacks(networkAgent, ConnectivityManager.CALLBACK_IP_CHANGED); } @@ -7149,7 +7150,9 @@ public class ConnectivityService extends IConnectivityManager.Stub networkAgent.networkMonitor().setAcceptPartialConnectivity(); } networkAgent.networkMonitor().notifyNetworkConnected( - networkAgent.linkProperties, networkAgent.networkCapabilities); + new LinkProperties(networkAgent.linkProperties, + true /* parcelSensitiveFields */), + networkAgent.networkCapabilities); scheduleUnvalidatedPrompt(networkAgent); // Whether a particular NetworkRequest listen should cause signal strength thresholds to