Merge "Minor changes and more cts tests for DnsResolver"
am: db7e161d85
Change-Id: Iea4fcc93ec15a4680bb27f3a51cb896d27513722
This commit is contained in:
@@ -17,16 +17,16 @@
|
|||||||
package android.net.cts;
|
package android.net.cts;
|
||||||
|
|
||||||
import static android.net.DnsResolver.CLASS_IN;
|
import static android.net.DnsResolver.CLASS_IN;
|
||||||
import static android.net.DnsResolver.TYPE_A;
|
|
||||||
import static android.net.DnsResolver.TYPE_AAAA;
|
|
||||||
import static android.net.DnsResolver.FLAG_NO_CACHE_LOOKUP;
|
import static android.net.DnsResolver.FLAG_NO_CACHE_LOOKUP;
|
||||||
|
import static android.net.DnsResolver.TYPE_AAAA;
|
||||||
|
|
||||||
|
import android.annotation.NonNull;
|
||||||
import android.content.Context;
|
import android.content.Context;
|
||||||
import android.net.ConnectivityManager;
|
import android.net.ConnectivityManager;
|
||||||
|
import android.net.DnsPacket;
|
||||||
import android.net.DnsResolver;
|
import android.net.DnsResolver;
|
||||||
import android.net.Network;
|
import android.net.Network;
|
||||||
import android.net.NetworkCapabilities;
|
import android.net.NetworkCapabilities;
|
||||||
import android.net.NetworkUtils;
|
|
||||||
import android.os.Handler;
|
import android.os.Handler;
|
||||||
import android.os.Looper;
|
import android.os.Looper;
|
||||||
import android.system.ErrnoException;
|
import android.system.ErrnoException;
|
||||||
@@ -35,14 +35,18 @@ import android.util.Log;
|
|||||||
|
|
||||||
import java.net.InetAddress;
|
import java.net.InetAddress;
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
|
import java.util.List;
|
||||||
import java.util.concurrent.CountDownLatch;
|
import java.util.concurrent.CountDownLatch;
|
||||||
import java.util.concurrent.TimeUnit;
|
import java.util.concurrent.TimeUnit;
|
||||||
|
import java.util.concurrent.atomic.AtomicReference;
|
||||||
|
|
||||||
public class DnsResolverTest extends AndroidTestCase {
|
public class DnsResolverTest extends AndroidTestCase {
|
||||||
private static final String TAG = "DnsResolverTest";
|
private static final String TAG = "DnsResolverTest";
|
||||||
private static final char[] HEX_CHARS = {
|
private static final char[] HEX_CHARS = {
|
||||||
'0', '1', '2', '3', '4', '5', '6', '7', '8', '9', 'A', 'B', 'C', 'D', 'E', 'F'
|
'0', '1', '2', '3', '4', '5', '6', '7', '8', '9', 'A', 'B', 'C', 'D', 'E', 'F'
|
||||||
};
|
};
|
||||||
|
static final int TIMEOUT_MS = 12_000;
|
||||||
|
|
||||||
private ConnectivityManager mCM;
|
private ConnectivityManager mCM;
|
||||||
private Handler mHandler;
|
private Handler mHandler;
|
||||||
private DnsResolver mDns;
|
private DnsResolver mDns;
|
||||||
@@ -54,7 +58,7 @@ public class DnsResolverTest extends AndroidTestCase {
|
|||||||
mDns = DnsResolver.getInstance();
|
mDns = DnsResolver.getInstance();
|
||||||
}
|
}
|
||||||
|
|
||||||
private static String bytesArrayToHexString(byte[] bytes) {
|
private static String byteArrayToHexString(byte[] bytes) {
|
||||||
char[] hexChars = new char[bytes.length * 2];
|
char[] hexChars = new char[bytes.length * 2];
|
||||||
for (int i = 0; i < bytes.length; ++i) {
|
for (int i = 0; i < bytes.length; ++i) {
|
||||||
int b = bytes[i] & 0xFF;
|
int b = bytes[i] & 0xFF;
|
||||||
@@ -77,85 +81,188 @@ public class DnsResolverTest extends AndroidTestCase {
|
|||||||
|
|
||||||
assertTrue(
|
assertTrue(
|
||||||
"This test requires that at least one network be connected. " +
|
"This test requires that at least one network be connected. " +
|
||||||
"Please ensure that the device is connected to a network.",
|
"Please ensure that the device is connected to a network.",
|
||||||
testableNetworks.size() >= 1);
|
testableNetworks.size() >= 1);
|
||||||
return testableNetworks.toArray(new Network[0]);
|
return testableNetworks.toArray(new Network[0]);
|
||||||
}
|
}
|
||||||
|
|
||||||
public void testInetAddressQuery() throws ErrnoException {
|
public void testInetAddressQuery() throws ErrnoException {
|
||||||
|
final String dname = "www.google.com";
|
||||||
|
final String msg = "InetAddress query " + dname;
|
||||||
for (Network network : getTestableNetworks()) {
|
for (Network network : getTestableNetworks()) {
|
||||||
CountDownLatch latch = new CountDownLatch(1);
|
final CountDownLatch latch = new CountDownLatch(1);
|
||||||
final int TIMEOUT_MS = 5_000;
|
final AtomicReference<List<InetAddress>> answers = new AtomicReference<>();
|
||||||
final String dname = "www.google.com";
|
|
||||||
|
|
||||||
mDns.query(network, dname, FLAG_NO_CACHE_LOOKUP, mHandler, answerList -> {
|
mDns.query(network, dname, FLAG_NO_CACHE_LOOKUP, mHandler, answerList -> {
|
||||||
if (answerList.size() != 0) {
|
answers.set(answerList);
|
||||||
latch.countDown();
|
|
||||||
for (InetAddress addr : answerList) {
|
for (InetAddress addr : answerList) {
|
||||||
Log.e(TAG, "Reported addr:" + addr.toString());
|
Log.d(TAG, "Reported addr:" + addr.toString());
|
||||||
}
|
}
|
||||||
|
latch.countDown();
|
||||||
}
|
}
|
||||||
}
|
|
||||||
);
|
);
|
||||||
String msg = "InetAddress query " + dname + " but no valid answer after "
|
|
||||||
+ TIMEOUT_MS + "ms.";
|
|
||||||
try {
|
try {
|
||||||
assertTrue(msg, latch.await(TIMEOUT_MS, TimeUnit.MILLISECONDS));
|
assertTrue(msg + " but no valid answer after " + TIMEOUT_MS + "ms.",
|
||||||
} catch (InterruptedException e) {}
|
latch.await(TIMEOUT_MS, TimeUnit.MILLISECONDS));
|
||||||
|
assertGreaterThan(msg + " returned 0 result", answers.get().size(), 0);
|
||||||
|
} catch (InterruptedException e) {
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
static private void assertGreaterThan(String msg, int a, int b) {
|
||||||
|
assertTrue(msg + ": " + a + " > " + b, a > b);
|
||||||
|
}
|
||||||
|
|
||||||
|
static private void assertValidAnswer(String msg, @NonNull DnsAnswer ans) {
|
||||||
|
// Check rcode field.(0, No error condition).
|
||||||
|
assertTrue(msg + " Response error, rcode: " + ans.getRcode(), ans.getRcode() == 0);
|
||||||
|
// Check answer counts.
|
||||||
|
assertTrue(msg + " No answer found", ans.getANCount() > 0);
|
||||||
|
// Check question counts.
|
||||||
|
assertTrue(msg + " No question found", ans.getQDCount() > 0);
|
||||||
|
}
|
||||||
|
|
||||||
|
static private void assertValidEmptyAnswer(String msg, @NonNull DnsAnswer ans) {
|
||||||
|
// Check rcode field.(0, No error condition).
|
||||||
|
assertTrue(msg + " Response error, rcode: " + ans.getRcode(), ans.getRcode() == 0);
|
||||||
|
// Check answer counts. Expect 0 answer.
|
||||||
|
assertTrue(msg + " Not an empty answer", ans.getANCount() == 0);
|
||||||
|
// Check question counts.
|
||||||
|
assertTrue(msg + " No question found", ans.getQDCount() > 0);
|
||||||
|
}
|
||||||
|
|
||||||
|
private class DnsAnswer extends DnsPacket {
|
||||||
|
DnsAnswer(@NonNull byte[] data) throws ParseException {
|
||||||
|
super(data);
|
||||||
|
// Check QR field.(query (0), or a response (1)).
|
||||||
|
if ((mHeader.flags & (1 << 15)) == 0) {
|
||||||
|
throw new ParseException("Not an answer packet");
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
int getRcode() {
|
||||||
|
return mHeader.rcode;
|
||||||
|
}
|
||||||
|
int getANCount(){
|
||||||
|
return mHeader.getRecordCount(ANSECTION);
|
||||||
|
}
|
||||||
|
int getQDCount(){
|
||||||
|
return mHeader.getRecordCount(QDSECTION);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
public void testRawQuery() throws ErrnoException {
|
public void testRawQuery() throws ErrnoException {
|
||||||
|
final String dname = "www.google.com";
|
||||||
|
final String msg = "Raw query " + dname;
|
||||||
for (Network network : getTestableNetworks()) {
|
for (Network network : getTestableNetworks()) {
|
||||||
CountDownLatch latch = new CountDownLatch(1);
|
final CountDownLatch latch = new CountDownLatch(1);
|
||||||
final int TIMEOUT_MS = 5_000;
|
mDns.query(network, dname, CLASS_IN, TYPE_AAAA, FLAG_NO_CACHE_LOOKUP, mHandler,
|
||||||
final String dname = "www.google.com";
|
answer -> {
|
||||||
|
if (answer == null) {
|
||||||
mDns.query(network, dname, CLASS_IN, TYPE_AAAA, FLAG_NO_CACHE_LOOKUP, mHandler, answer -> {
|
fail(msg + " no answer returned");
|
||||||
if (answer != null) {
|
}
|
||||||
latch.countDown();
|
try {
|
||||||
Log.e(TAG, "Reported blob:" + bytesArrayToHexString(answer));
|
assertValidAnswer(msg, new DnsAnswer(answer));
|
||||||
|
Log.d(TAG, "Reported blob:" + byteArrayToHexString(answer));
|
||||||
|
latch.countDown();
|
||||||
|
} catch (DnsPacket.ParseException e) {
|
||||||
|
fail(msg + e.getMessage());
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
|
||||||
);
|
);
|
||||||
String msg = "Raw query " + dname + " but no valid answer after " + TIMEOUT_MS + "ms.";
|
|
||||||
try {
|
try {
|
||||||
assertTrue(msg, latch.await(TIMEOUT_MS, TimeUnit.MILLISECONDS));
|
assertTrue(msg + " but no answer after " + TIMEOUT_MS + "ms.",
|
||||||
} catch (InterruptedException e) {}
|
latch.await(TIMEOUT_MS, TimeUnit.MILLISECONDS));
|
||||||
|
} catch (InterruptedException e) {
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
public void testRawQueryWithBlob() throws ErrnoException {
|
public void testRawQueryWithBlob() throws ErrnoException {
|
||||||
|
final byte[] blob = new byte[]{
|
||||||
|
/* Header */
|
||||||
|
0x55, 0x66, /* Transaction ID */
|
||||||
|
0x01, 0x00, /* Flags */
|
||||||
|
0x00, 0x01, /* Questions */
|
||||||
|
0x00, 0x00, /* Answer RRs */
|
||||||
|
0x00, 0x00, /* Authority RRs */
|
||||||
|
0x00, 0x00, /* Additional RRs */
|
||||||
|
/* Queries */
|
||||||
|
0x03, 0x77, 0x77, 0x77, 0x06, 0x67, 0x6F, 0x6F, 0x67, 0x6c, 0x65,
|
||||||
|
0x03, 0x63, 0x6f, 0x6d, 0x00, /* Name */
|
||||||
|
0x00, 0x01, /* Type */
|
||||||
|
0x00, 0x01 /* Class */
|
||||||
|
};
|
||||||
|
final String msg = "Raw query with blob " + byteArrayToHexString(blob);
|
||||||
for (Network network : getTestableNetworks()) {
|
for (Network network : getTestableNetworks()) {
|
||||||
CountDownLatch latch = new CountDownLatch(1);
|
final CountDownLatch latch = new CountDownLatch(1);
|
||||||
final int TIMEOUT_MS = 5_000;
|
|
||||||
final byte[] blob = new byte[] {
|
|
||||||
/* Header */
|
|
||||||
0x55, 0x66, /* Transaction ID */
|
|
||||||
0x01, 0x00, /* Flags */
|
|
||||||
0x00, 0x01, /* Questions */
|
|
||||||
0x00, 0x00, /* Answer RRs */
|
|
||||||
0x00, 0x00, /* Authority RRs */
|
|
||||||
0x00, 0x00, /* Additional RRs */
|
|
||||||
/* Queries */
|
|
||||||
0x03, 0x77, 0x77, 0x77, 0x06, 0x67, 0x6F, 0x6F, 0x67, 0x6c, 0x65,
|
|
||||||
0x03, 0x63, 0x6f, 0x6d, 0x00, /* Name */
|
|
||||||
0x00, 0x01, /* Type */
|
|
||||||
0x00, 0x01 /* Class */
|
|
||||||
};
|
|
||||||
|
|
||||||
mDns.query(network, blob, FLAG_NO_CACHE_LOOKUP, mHandler, answer -> {
|
mDns.query(network, blob, FLAG_NO_CACHE_LOOKUP, mHandler, answer -> {
|
||||||
if (answer != null) {
|
if (answer == null) {
|
||||||
latch.countDown();
|
fail(msg + " no answer returned");
|
||||||
Log.e(TAG, "Reported blob:" + bytesArrayToHexString(answer));
|
}
|
||||||
|
try {
|
||||||
|
assertValidAnswer(msg, new DnsAnswer(answer));
|
||||||
|
Log.d(TAG, "Reported blob:" + byteArrayToHexString(answer));
|
||||||
|
latch.countDown();
|
||||||
|
} catch (DnsPacket.ParseException e) {
|
||||||
|
fail(msg + e.getMessage());
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
|
||||||
);
|
);
|
||||||
String msg = "Raw query with blob " + bytesArrayToHexString(blob) +
|
|
||||||
" but no valid answer after " + TIMEOUT_MS + "ms.";
|
|
||||||
try {
|
try {
|
||||||
assertTrue(msg, latch.await(TIMEOUT_MS, TimeUnit.MILLISECONDS));
|
assertTrue(msg + " but no answer after " + TIMEOUT_MS + "ms.",
|
||||||
} catch (InterruptedException e) {}
|
latch.await(TIMEOUT_MS, TimeUnit.MILLISECONDS));
|
||||||
|
} catch (InterruptedException e) {
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
public void testEmptyQuery() throws ErrnoException {
|
||||||
|
final String dname = "";
|
||||||
|
final String msg = "Raw query empty dname(ROOT)";
|
||||||
|
for (Network network : getTestableNetworks()) {
|
||||||
|
final CountDownLatch latch = new CountDownLatch(1);
|
||||||
|
mDns.query(network, dname, CLASS_IN, TYPE_AAAA, FLAG_NO_CACHE_LOOKUP, mHandler,
|
||||||
|
answer -> {
|
||||||
|
if (answer == null) {
|
||||||
|
fail(msg + " no answer returned");
|
||||||
|
}
|
||||||
|
try {
|
||||||
|
// Except no answer record because of querying with empty dname(ROOT)
|
||||||
|
assertValidEmptyAnswer(msg, new DnsAnswer(answer));
|
||||||
|
latch.countDown();
|
||||||
|
} catch (DnsPacket.ParseException e) {
|
||||||
|
fail(msg + e.getMessage());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
);
|
||||||
|
try {
|
||||||
|
assertTrue(msg + " but no answer after " + TIMEOUT_MS + "ms.",
|
||||||
|
latch.await(TIMEOUT_MS, TimeUnit.MILLISECONDS));
|
||||||
|
} catch (InterruptedException e) {
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
public void testNXQuery() throws ErrnoException {
|
||||||
|
final String dname = "test1-nx.metric.gstatic.com";
|
||||||
|
final String msg = "InetAddress query " + dname;
|
||||||
|
for (Network network : getTestableNetworks()) {
|
||||||
|
final CountDownLatch latch = new CountDownLatch(1);
|
||||||
|
mDns.query(network, dname, FLAG_NO_CACHE_LOOKUP, mHandler, answerList -> {
|
||||||
|
if (answerList.size() == 0) {
|
||||||
|
latch.countDown();
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
fail(msg + " but get valid answers");
|
||||||
|
}
|
||||||
|
);
|
||||||
|
try {
|
||||||
|
assertTrue(msg + " but no answer after " + TIMEOUT_MS + "ms.",
|
||||||
|
latch.await(TIMEOUT_MS, TimeUnit.MILLISECONDS));
|
||||||
|
} catch (InterruptedException e) {
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user