summaryrefslogtreecommitdiff
path: root/service
diff options
context:
space:
mode:
authorNate Jiang <qiangjiang@google.com>2019-07-08 17:53:07 -0700
committerNate Jiang <qiangjiang@google.com>2019-07-09 01:22:06 +0000
commit97bbfd5808b868f1b4c6af86a19ce821c13e872e (patch)
tree291569e9f9382d26a54d83c9155394e8bbf15c98 /service
parent17aa8035cec2802ea19464fe76d9f31ef4bc6af1 (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.java7
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!?");