updated tests

tests/testsuite.c

@@ -1,6 +1,7 @@
 #include <stdint.h>
 #include <stdio.h>
 
+#define INTEST 1
 
 #include "../tinytest/tinytest.h"
 #include "../user/dns_util.h"
@@ -25,43 +26,53 @@ unsigned char dns_smallresponse[] = {
 void test_dnsname_no_compression()
 {
     // no dns compression, name compare simply and it works
-    ASSERT_EQUALS(0, dns_compare_name("a.example.com", dns_response, sizeof(dns_response), 12));
+    ASSERT_EQUALS(0, bkdns_compare_name("a.example.com", dns_response, sizeof(dns_response), 12));
     // no dns compression, name compare simply and it works. Trailing dot ok.
-    ASSERT_EQUALS(0, dns_compare_name("a.example.com.", dns_response, sizeof(dns_response), 12));
+    ASSERT_EQUALS(0, bkdns_compare_name("a.example.com.", dns_response, sizeof(dns_response), 12));
     // pointer to the wrong part of the name, shouldn't compare successfully, we pointed at example.com
-    ASSERT_EQUALS(1, dns_compare_name("a.example.com", dns_response, sizeof(dns_response), 14));
+    ASSERT_EQUALS(1, bkdns_compare_name("a.example.com", dns_response, sizeof(dns_response), 14));
     // pointer to the start of the message, but is only the start of the hostname, so this should fail
-    ASSERT_EQUALS(1, dns_compare_name("a.example", dns_response, sizeof(dns_response), 12));
+    ASSERT_EQUALS(1, bkdns_compare_name("a.example", dns_response, sizeof(dns_response), 12));
 
     // one segment too long....
-    ASSERT_EQUALS(1, dns_compare_name("a.examplea.com.", dns_response, sizeof(dns_response), 12));
+    ASSERT_EQUALS(1, bkdns_compare_name("a.examplea.com.", dns_response, sizeof(dns_response), 12));
     // one segment too short....
-    ASSERT_EQUALS(1, dns_compare_name("a.exampl.com.", dns_response, sizeof(dns_response), 12));
+    ASSERT_EQUALS(1, bkdns_compare_name("a.exampl.com.", dns_response, sizeof(dns_response), 12));
 }
 
 void test_malicious_input_rejected() {
     // no infinite loop....
-    ASSERT_EQUALS(1, dns_compare_name(".", evil_dns_response_selfreference, sizeof(evil_dns_response_selfreference), 2));
+    ASSERT_EQUALS(1, bkdns_compare_name(".", evil_dns_response_selfreference, sizeof(evil_dns_response_selfreference), 2));
 }
 
 void test_bounds_truncate() {
-    ASSERT_EQUALS(0, dns_compare_name("test.com", dns_smallresponse, sizeof(dns_smallresponse), 4));
-    ASSERT_EQUALS(1, dns_compare_name("test.com", dns_smallresponse, sizeof(dns_smallresponse) - 2, 4));
-    ASSERT_EQUALS(1, dns_compare_name("test.com", dns_smallresponse, sizeof(dns_smallresponse) - 1, 4));
+    ASSERT_EQUALS(0, bkdns_compare_name("test.com", dns_smallresponse, sizeof(dns_smallresponse), 4));
+    ASSERT_EQUALS(1, bkdns_compare_name("test.com", dns_smallresponse, sizeof(dns_smallresponse) - 2, 4));
+    ASSERT_EQUALS(1, bkdns_compare_name("test.com", dns_smallresponse, sizeof(dns_smallresponse) - 1, 4));
 
 }
 
 void test_dnsname_with_compression()
 {
     // pointer to the part of the message that used DNS compression properly. False positive in stock LWIP...
-    ASSERT_EQUALS(0, dns_compare_name("b.example.com", dns_response, sizeof(dns_response), 27));
+    ASSERT_EQUALS(0, bkdns_compare_name("b.example.com", dns_response, sizeof(dns_response), 27));
     // it should work when case insensitive also, see RFC 4343 sec 4.1
-    ASSERT_EQUALS(0, dns_compare_name("b.Example.com", dns_response, sizeof(dns_response), 27));
+    ASSERT_EQUALS(0, bkdns_compare_name("b.Example.com", dns_response, sizeof(dns_response), 27));
 
     // below here is where things failed before my substantive changes.  This code code from lwip had
     // anything that has dns compression matching everything as soon as the compression kicks in..
     // pointer to the compressed a.example.com in the message, but we aren't comparing that
-    ASSERT_EQUALS(1, dns_compare_name("b.bing.com", dns_response, sizeof(dns_response), 27));
+    ASSERT_EQUALS(1, bkdns_compare_name("b.bing.com", dns_response, sizeof(dns_response), 27));
+}
+
+// query was for _thingsbus._tcp.home.truct.org
+// result is frigg.truct.org srv record
+unsigned char example_response[] = "\x00\x1b\x81\x80\x00\x01\x00\x01\x00\x00\x00\x00\n_thingsbus\x04_tcp"\
+                                   "\x04home\x05truct\x03org\x00\x00!\x00\x01\xc0\x0c\x00!\x00\x01\x00\x00"\
+                                   "T_\x00\x17\x00\x00\x00\x01\x1f\x12\x05frigg\x05truct\x03org\x00";
+
+void test_parse_srv_record() {
+    // TODO actually write this
 }
 
 /* test runner */