From 66d428e3c04eed9a20e0aa80c6e687aee622fc24 Mon Sep 17 00:00:00 2001 From: David Su Date: Fri, 28 Aug 2020 13:09:35 -0700 Subject: LINK_CONFIGURATION_CHANGED broadcast: stop including EXTRA_LINK_PROPERTIES This extra is unused, and potentially exposes sensitive information that can be accessed without any permissions. Bug: 162246414 Test: atest FrameworksWifiTests Change-Id: Iae0339790ac9a664c6e73bc4254f1176c6d54f3b Merged-In: Iae0339790ac9a664c6e73bc4254f1176c6d54f3b --- service/java/com/android/server/wifi/ClientModeImpl.java | 1 - 1 file changed, 1 deletion(-) diff --git a/service/java/com/android/server/wifi/ClientModeImpl.java b/service/java/com/android/server/wifi/ClientModeImpl.java index 95983096a..88ce0eda0 100644 --- a/service/java/com/android/server/wifi/ClientModeImpl.java +++ b/service/java/com/android/server/wifi/ClientModeImpl.java @@ -2563,7 +2563,6 @@ public class ClientModeImpl extends StateMachine { private void sendLinkConfigurationChangedBroadcast() { Intent intent = new Intent(WifiManager.ACTION_LINK_CONFIGURATION_CHANGED); intent.addFlags(Intent.FLAG_RECEIVER_REGISTERED_ONLY_BEFORE_BOOT); - intent.putExtra(WifiManager.EXTRA_LINK_PROPERTIES, new LinkProperties(mLinkProperties)); mContext.sendBroadcastAsUser(intent, UserHandle.ALL); } -- cgit v1.2.3