From 6d5e944ba5bbd98c6ff9f29104fc2537d3a8ce8a Mon Sep 17 00:00:00 2001 From: Chalard Jean Date: Fri, 9 Oct 2020 17:04:02 +0900 Subject: [PATCH] Don't let NAW.mNetworkInfo escape Thankfully this has no users. This agent needs to be migrated to the modern API for NetworkAgent, which does not have NetworkInfo at all so this would be a hindrance Test: FrameworksNetTests NetworkStackTests Change-Id: I091a6ca8cdf18860f23832ae1008fa02cf0472e8 --- .../util/com/android/server/NetworkAgentWrapper.java | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/tests/net/integration/util/com/android/server/NetworkAgentWrapper.java b/tests/net/integration/util/com/android/server/NetworkAgentWrapper.java index c895420157d70..85704d0336340 100644 --- a/tests/net/integration/util/com/android/server/NetworkAgentWrapper.java +++ b/tests/net/integration/util/com/android/server/NetworkAgentWrapper.java @@ -213,7 +213,7 @@ public class NetworkAgentWrapper implements TestableNetworkCallback.HasNetwork { public void connect() { assertNotEquals("MockNetworkAgents can only be connected once", - getNetworkInfo().getDetailedState(), NetworkInfo.DetailedState.CONNECTED); + mNetworkInfo.getDetailedState(), NetworkInfo.DetailedState.CONNECTED); mNetworkInfo.setDetailedState(NetworkInfo.DetailedState.CONNECTED, null, null); mNetworkAgent.sendNetworkInfo(mNetworkInfo); } @@ -268,10 +268,6 @@ public class NetworkAgentWrapper implements TestableNetworkCallback.HasNetwork { return mNetworkAgent; } - public NetworkInfo getNetworkInfo() { - return mNetworkInfo; - } - public NetworkCapabilities getNetworkCapabilities() { return mNetworkCapabilities; }