summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRoshan Pius <rpius@google.com>2020-03-30 16:19:41 -0700
committerRoshan Pius <rpius@google.com>2020-03-30 22:42:39 -0700
commitb98ca721f6efed2fe85c07153e05e6410e741af4 (patch)
tree5b0a02b267673378135dbc55274f5023f9eaaa1e
parent6957462276b7ce2c005418eb1692d6f22960d380 (diff)
ClientModeImpl: Add a wtf for duplicate NetworkAgent registration
We invoke unregister & reset mNetworkAgent in handleNetworkDisconnect(). So, this should never be non-null when we create a new NetworkAgent instance for a new connection. Bug: 151514192 Test: Manual log verification: Verified that the log message is not printed for regular wifi switching. Change-Id: I0187e50119c8fa0cdb2e900da7e60fb8a3f653f9
-rw-r--r--service/java/com/android/server/wifi/ClientModeImpl.java4
1 files changed, 4 insertions, 0 deletions
diff --git a/service/java/com/android/server/wifi/ClientModeImpl.java b/service/java/com/android/server/wifi/ClientModeImpl.java
index 270af89d1..588564ed9 100644
--- a/service/java/com/android/server/wifi/ClientModeImpl.java
+++ b/service/java/com/android/server/wifi/ClientModeImpl.java
@@ -4636,6 +4636,10 @@ public class ClientModeImpl extends StateMachine {
.build();
final NetworkCapabilities nc = getCapabilities(getCurrentWifiConfiguration());
synchronized (mNetworkAgentLock) {
+ // This should never happen.
+ if (mNetworkAgent != null) {
+ Log.wtf(TAG, "mNetworkAgent is not null: " + mNetworkAgent);
+ }
mNetworkAgent = new WifiNetworkAgent(mContext, getHandler().getLooper(),
"WifiNetworkAgent", nc, mLinkProperties, 60, naConfig,
mNetworkFactory.getProvider());