diff options
author | Nate Jiang <qiangjiang@google.com> | 2019-07-08 17:53:07 -0700 |
---|---|---|
committer | Nate Jiang <qiangjiang@google.com> | 2019-07-09 01:22:06 +0000 |
commit | 97bbfd5808b868f1b4c6af86a19ce821c13e872e (patch) | |
tree | 291569e9f9382d26a54d83c9155394e8bbf15c98 /service | |
parent | 17aa8035cec2802ea19464fe76d9f31ef4bc6af1 (diff) |
[AWARE] create interface before connect success
After restart, when first time using Aware to create NDP, onConnectSuccess send before all interface created. That may cause NDP unavailable.
Change the order of create interface and onConnectSuccess.
Bug: 124653166
Test: atest android.net.wifi
Test: atest com.android.server.wifi
Test: ACTS DataPathTest
Change-Id: I96d1eb4da51046b6d6928287a439965c8c50bff6
Diffstat (limited to 'service')
-rw-r--r-- | service/java/com/android/server/wifi/aware/WifiAwareStateManager.java | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/service/java/com/android/server/wifi/aware/WifiAwareStateManager.java b/service/java/com/android/server/wifi/aware/WifiAwareStateManager.java index a117ecc38..072788081 100644 --- a/service/java/com/android/server/wifi/aware/WifiAwareStateManager.java +++ b/service/java/com/android/server/wifi/aware/WifiAwareStateManager.java @@ -2577,6 +2577,10 @@ public class WifiAwareStateManager implements WifiAwareShellCommand.DelegatedShe } if (completedCommand.arg1 == COMMAND_TYPE_CONNECT) { + if (mCurrentAwareConfiguration == null) { // enabled (as opposed to re-configured) + createAllDataPathInterfaces(); + } + Bundle data = completedCommand.getData(); int clientId = completedCommand.arg2; @@ -2615,9 +2619,6 @@ public class WifiAwareStateManager implements WifiAwareShellCommand.DelegatedShe return; } - if (mCurrentAwareConfiguration == null) { // enabled (as opposed to re-configured) - createAllDataPathInterfaces(); - } mCurrentAwareConfiguration = mergeConfigRequests(null); if (mCurrentAwareConfiguration == null) { Log.wtf(TAG, "onConfigCompletedLocal: got a null merged configuration after config!?"); |