diff --git a/nexus/WifiNetwork.cpp b/nexus/WifiNetwork.cpp index 1f53a20f3..818b91d95 100644 --- a/nexus/WifiNetwork.cpp +++ b/nexus/WifiNetwork.cpp @@ -551,6 +551,20 @@ int WifiNetwork::setAllowedGroupCiphers(uint32_t mask) { } int WifiNetwork::setEnabled(bool enabled) { + + if (enabled) { + if (getPriority() == -1) { + LOGE("Cannot enable network when priority is not set"); + errno = EAGAIN; + return -1; + } + if (getAllowedKeyManagement() == KeyManagementMask::UNKNOWN) { + LOGE("Cannot enable network when KeyManagement is not set"); + errno = EAGAIN; + return -1; + } + } + if (mSuppl->enableNetwork(mNetid, enabled)) return -1; diff --git a/nexus/WifiNetwork.h b/nexus/WifiNetwork.h index bdffa8be6..360ccc28c 100644 --- a/nexus/WifiNetwork.h +++ b/nexus/WifiNetwork.h @@ -23,10 +23,11 @@ class KeyManagementMask { public: - static const uint32_t NONE = 0; - static const uint32_t WPA_PSK = 0x01; - static const uint32_t WPA_EAP = 0x02; - static const uint32_t IEEE8021X = 0x04; + static const uint32_t UNKNOWN = 0; + static const uint32_t NONE = 0x01; + static const uint32_t WPA_PSK = 0x02; + static const uint32_t WPA_EAP = 0x04; + static const uint32_t IEEE8021X = 0x08; static const uint32_t ALL = WPA_PSK | WPA_EAP | IEEE8021X; };