diff options
author | Lorenzo Colitti <lorenzo@google.com> | 2014-07-31 00:48:01 +0900 |
---|---|---|
committer | Lorenzo Colitti <lorenzo@google.com> | 2014-08-19 11:59:43 -0700 |
commit | 0a82e80073e193725a9d4c84a93db8a04b2456b9 (patch) | |
tree | 0fc1bb0fab2818df4175215e88ac52372e20bc66 /core/jni/android_net_NetUtils.cpp | |
parent | 3a6eba01c48363f675090bd8e0f853a7fbf2165d (diff) | |
download | frameworks_base-0a82e80073e193725a9d4c84a93db8a04b2456b9.zip frameworks_base-0a82e80073e193725a9d4c84a93db8a04b2456b9.tar.gz frameworks_base-0a82e80073e193725a9d4c84a93db8a04b2456b9.tar.bz2 |
Stop using LinkProperties for static configuration.
LinkProperties can represent way more complicated configurations
than what we can actually apply to interfaces. This makes it
error-prone to use it to represent static configuration, both
when trying to apply configuration coming from LinkProperties
and when trying to save configuration from current
LinkProperties.
Instead, move static configuration (IPv4 only, since we don't
support static IPv6 configuration) into a separate
StaticIpConfiguration class.
Bug: 16114392
Bug: 16893413
Change-Id: Ib33f35c004e30b6067bb20235ffa43c247d174df
Diffstat (limited to 'core/jni/android_net_NetUtils.cpp')
-rw-r--r-- | core/jni/android_net_NetUtils.cpp | 24 |
1 files changed, 8 insertions, 16 deletions
diff --git a/core/jni/android_net_NetUtils.cpp b/core/jni/android_net_NetUtils.cpp index 32cf286..5bd38f3 100644 --- a/core/jni/android_net_NetUtils.cpp +++ b/core/jni/android_net_NetUtils.cpp @@ -69,9 +69,8 @@ namespace android { */ static struct fieldIds { jmethodID clear; - jmethodID setInterfaceName; - jmethodID addLinkAddress; - jmethodID addGateway; + jmethodID setIpAddress; + jmethodID setGateway; jmethodID addDns; jmethodID setDomains; jmethodID setServerAddress; @@ -130,21 +129,16 @@ static jboolean android_net_utils_runDhcpCommon(JNIEnv* env, jobject clazz, jstr if (result == 0) { env->CallVoidMethod(dhcpResults, dhcpResultsFieldIds.clear); - // set mIfaceName - // dhcpResults->setInterfaceName(ifname) - env->CallVoidMethod(dhcpResults, dhcpResultsFieldIds.setInterfaceName, ifname); - // set the linkAddress // dhcpResults->addLinkAddress(inetAddress, prefixLength) - result = env->CallBooleanMethod(dhcpResults, dhcpResultsFieldIds.addLinkAddress, + result = env->CallBooleanMethod(dhcpResults, dhcpResultsFieldIds.setIpAddress, env->NewStringUTF(ipaddr), prefixLength); } if (result == 0) { // set the gateway - // dhcpResults->addGateway(gateway) result = env->CallBooleanMethod(dhcpResults, - dhcpResultsFieldIds.addGateway, env->NewStringUTF(gateway)); + dhcpResultsFieldIds.setGateway, env->NewStringUTF(gateway)); } if (result == 0) { @@ -279,12 +273,10 @@ int register_android_net_NetworkUtils(JNIEnv* env) LOG_FATAL_IF(dhcpResultsClass == NULL, "Unable to find class android/net/DhcpResults"); dhcpResultsFieldIds.clear = env->GetMethodID(dhcpResultsClass, "clear", "()V"); - dhcpResultsFieldIds.setInterfaceName = - env->GetMethodID(dhcpResultsClass, "setInterfaceName", "(Ljava/lang/String;)V"); - dhcpResultsFieldIds.addLinkAddress = - env->GetMethodID(dhcpResultsClass, "addLinkAddress", "(Ljava/lang/String;I)Z"); - dhcpResultsFieldIds.addGateway = - env->GetMethodID(dhcpResultsClass, "addGateway", "(Ljava/lang/String;)Z"); + dhcpResultsFieldIds.setIpAddress = + env->GetMethodID(dhcpResultsClass, "setIpAddress", "(Ljava/lang/String;I)Z"); + dhcpResultsFieldIds.setGateway = + env->GetMethodID(dhcpResultsClass, "setGateway", "(Ljava/lang/String;)Z"); dhcpResultsFieldIds.addDns = env->GetMethodID(dhcpResultsClass, "addDns", "(Ljava/lang/String;)Z"); dhcpResultsFieldIds.setDomains = |