diff options
author | Roshan Pius <rpius@google.com> | 2020-03-30 16:19:41 -0700 |
---|---|---|
committer | Roshan Pius <rpius@google.com> | 2020-03-30 22:42:39 -0700 |
commit | b98ca721f6efed2fe85c07153e05e6410e741af4 (patch) | |
tree | 5b0a02b267673378135dbc55274f5023f9eaaa1e | |
parent | 6957462276b7ce2c005418eb1692d6f22960d380 (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.java | 4 |
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()); |