diff options
| author | Rebecca Silberstein <silberst@google.com> | 2017-05-31 21:41:51 -0700 |
|---|---|---|
| committer | Rebecca Silberstein <silberst@google.com> | 2017-06-06 11:38:05 -0700 |
| commit | 1fdb6151dcc85c990483492e2d9cbd9b7c243164 (patch) | |
| tree | e73414c6d1a80d27079a8ee25bfee871df4a1707 | |
| parent | 1f6626d7488bba014b42bab8bbcd4720694df485 (diff) | |
| download | android_frameworks_opt_net_wifi-1fdb6151dcc85c990483492e2d9cbd9b7c243164.tar.gz android_frameworks_opt_net_wifi-1fdb6151dcc85c990483492e2d9cbd9b7c243164.tar.bz2 android_frameworks_opt_net_wifi-1fdb6151dcc85c990483492e2d9cbd9b7c243164.zip | |
WifiApConfigStore: set LOHS wifi config networkId
The WifiConfiguration object used to connect to a locally started LOHS
needs a networkId that signals to ConnectivityService that the
application wants to use the LOHS to talk to co-located devices.
Bug: 62076211
Test: frameworks/opt/net/wifi/tests/wifitests/runtests.sh -e class
com.android.server.wifi.WifiApConfigStoreTest
Test: frameworks/opt/net/wifi/tests/wifitests/runtests.sh -e class
com.android.server.wifi.WifiServiceImplTest
Test: manually tested with in-progress integration test
Change-Id: Ic316250184eae0f04dc0fb83efb9710d5d65f0af
| -rw-r--r-- | service/java/com/android/server/wifi/WifiApConfigStore.java | 1 | ||||
| -rw-r--r-- | tests/wifitests/src/com/android/server/wifi/WifiApConfigStoreTest.java | 2 |
2 files changed, 3 insertions, 0 deletions
diff --git a/service/java/com/android/server/wifi/WifiApConfigStore.java b/service/java/com/android/server/wifi/WifiApConfigStore.java index 850c5aebe..9c90bcf47 100644 --- a/service/java/com/android/server/wifi/WifiApConfigStore.java +++ b/service/java/com/android/server/wifi/WifiApConfigStore.java @@ -218,6 +218,7 @@ public class WifiApConfigStore { R.string.wifi_localhotspot_configure_ssid_default) + "_" + getRandomIntForDefaultSsid(); config.allowedKeyManagement.set(KeyMgmt.WPA2_PSK); + config.networkId = WifiConfiguration.LOCAL_ONLY_NETWORK_ID; String randomUUID = UUID.randomUUID().toString(); // first 12 chars from xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx config.preSharedKey = randomUUID.substring(0, 8) + randomUUID.substring(9, 13); diff --git a/tests/wifitests/src/com/android/server/wifi/WifiApConfigStoreTest.java b/tests/wifitests/src/com/android/server/wifi/WifiApConfigStoreTest.java index d4a3ff549..2d3b06695 100644 --- a/tests/wifitests/src/com/android/server/wifi/WifiApConfigStoreTest.java +++ b/tests/wifitests/src/com/android/server/wifi/WifiApConfigStoreTest.java @@ -202,5 +202,7 @@ public class WifiApConfigStoreTest { public void generateLocalOnlyHotspotConfigIsValid() { WifiConfiguration config = WifiApConfigStore.generateLocalOnlyHotspotConfig(mContext); verifyDefaultApConfig(config, TEST_DEFAULT_HOTSPOT_SSID); + // The LOHS config should also have a specific network id set - check that as well. + assertEquals(WifiConfiguration.LOCAL_ONLY_NETWORK_ID, config.networkId); } } |