diff options
author | Irfan Sheriff <isheriff@google.com> | 2011-01-27 11:05:04 -0800 |
---|---|---|
committer | Android (Google) Code Review <android-gerrit@google.com> | 2011-01-27 11:05:04 -0800 |
commit | e8e0ebbfd09214020cfbbaafaeb940a2e52985b3 (patch) | |
tree | d87cfced30acce2ab0dfd59fd9a3342431c80233 | |
parent | 6adcce8a5751b0afba86a55f600aa1f14a56d03c (diff) | |
parent | 68e18d23298c62b61844eb296d247df55d45f463 (diff) | |
download | frameworks_base-e8e0ebbfd09214020cfbbaafaeb940a2e52985b3.zip frameworks_base-e8e0ebbfd09214020cfbbaafaeb940a2e52985b3.tar.gz frameworks_base-e8e0ebbfd09214020cfbbaafaeb940a2e52985b3.tar.bz2 |
Merge "Fix the build"
-rw-r--r-- | core/java/android/net/DhcpInfoInternal.java | 26 | ||||
-rw-r--r-- | core/java/android/net/NetworkUtils.java | 4 |
2 files changed, 25 insertions, 5 deletions
diff --git a/core/java/android/net/DhcpInfoInternal.java b/core/java/android/net/DhcpInfoInternal.java index 7d9bd52..6e981df 100644 --- a/core/java/android/net/DhcpInfoInternal.java +++ b/core/java/android/net/DhcpInfoInternal.java @@ -16,6 +16,9 @@ package android.net; +import android.text.TextUtils; +import android.util.Log; + import java.net.InetAddress; import java.net.Inet4Address; import java.net.UnknownHostException; @@ -26,6 +29,7 @@ import java.net.UnknownHostException; * @hide */ public class DhcpInfoInternal { + private final static String TAG = "DhcpInfoInternal"; public String ipAddress; public String gateway; public int prefixLength; @@ -65,15 +69,31 @@ public class DhcpInfoInternal { } public LinkAddress makeLinkAddress() { + if (TextUtils.isEmpty(ipAddress)) { + Log.e(TAG, "makeLinkAddress with empty ipAddress"); + return null; + } return new LinkAddress(NetworkUtils.numericToInetAddress(ipAddress), prefixLength); } public LinkProperties makeLinkProperties() { LinkProperties p = new LinkProperties(); p.addLinkAddress(makeLinkAddress()); - p.setGateway(NetworkUtils.numericToInetAddress(gateway)); - p.addDns(NetworkUtils.numericToInetAddress(dns1)); - p.addDns(NetworkUtils.numericToInetAddress(dns2)); + if (TextUtils.isEmpty(gateway) == false) { + p.setGateway(NetworkUtils.numericToInetAddress(gateway)); + } else { + Log.e(TAG, "makeLinkProperties with empty gateway!"); + } + if (TextUtils.isEmpty(dns1) == false) { + p.addDns(NetworkUtils.numericToInetAddress(dns1)); + } else { + Log.e(TAG, "makeLinkProperties with empty dns1!"); + } + if (TextUtils.isEmpty(dns2) == false) { + p.addDns(NetworkUtils.numericToInetAddress(dns2)); + } else { + Log.e(TAG, "makeLinkProperties with empty dns2!"); + } return p; } diff --git a/core/java/android/net/NetworkUtils.java b/core/java/android/net/NetworkUtils.java index 1c48e7d..97f96da 100644 --- a/core/java/android/net/NetworkUtils.java +++ b/core/java/android/net/NetworkUtils.java @@ -162,8 +162,8 @@ public class NetworkUtils { // TODO - do this for real, using a hidden method on InetAddress that aborts // instead of doing dns step if (!InetAddress.isNumeric(addrString)) { - throw new IllegalArgumentException("numericToInetAddress with non numeric: " + - addrString); + throw new IllegalArgumentException("numericToInetAddress with non numeric: '" + + addrString + "'"); } try { return InetAddress.getByName(addrString); |