Multihoming fixed according to new LWIP implementation.

Author: tymoteuszblochmobica

TESTS/netsocket/tcp/main.cpp

@@ -194,10 +194,10 @@ static void test_failure_handler(const failure_t failure)
 
 
 Case cases[] = {
+    Case("TCPSOCKET_OPEN_LIMIT", TCPSOCKET_OPEN_LIMIT),
     Case("TCPSOCKET_ECHOTEST", TCPSOCKET_ECHOTEST),
     Case("TCPSOCKET_ECHOTEST_NONBLOCK", TCPSOCKET_ECHOTEST_NONBLOCK),
     Case("TCPSOCKET_OPEN_CLOSE_REPEAT", TCPSOCKET_OPEN_CLOSE_REPEAT),
-    Case("TCPSOCKET_OPEN_LIMIT", TCPSOCKET_OPEN_LIMIT),
     Case("TCPSOCKET_THREAD_PER_SOCKET_SAFETY", TCPSOCKET_THREAD_PER_SOCKET_SAFETY),
     Case("TCPSOCKET_CONNECT_INVALID", TCPSOCKET_CONNECT_INVALID),
     Case("TCPSOCKET_ECHOTEST_BURST", TCPSOCKET_ECHOTEST_BURST),

TESTS/network/multihoming/main.cpp

@@ -29,6 +29,8 @@
 #error [NOT_SUPPORTED] Both Wifi and Ethernet devices are required for multihoming tests.
 #endif
 
+#define STRING_VERIFY(str) (str != NULL ? str : "not supported")
+
 
 #include "greentea-client/test_env.h"
 #include "unity/unity.h"
@@ -95,13 +97,12 @@ static void _ifup()
             TEST_FAIL_MESSAGE("Wifi connection error!");
             return;
         }
-        wifi->get_interface_name(interface_name[interface_num]);
-        printf("MAC: %s\n", wifi->get_mac_address());
-        printf("IP: %s\n", wifi->get_ip_address());
-        printf("Netmask: %s\n", wifi->get_netmask());
-        printf("Gateway: %s\n", wifi->get_gateway());
+        printf("Wifi interface name: %s\n\n", STRING_VERIFY(wifi->get_interface_name(interface_name[interface_num])));
+        printf("MAC: %s\n", STRING_VERIFY(wifi->get_mac_address()));
+        printf("IP: %s\n", STRING_VERIFY(wifi->get_ip_address()));
+        printf("Netmask: %s\n", STRING_VERIFY(wifi->get_netmask()));
+        printf("Gateway: %s\n", STRING_VERIFY(wifi->get_gateway()));
         printf("RSSI: %d\n\n", wifi->get_rssi());
-        printf("Wifi interface name: %s\n\n", interface_name[interface_num]);
         interface_num++;
     } else {
         TEST_FAIL_MESSAGE("ERROR: No WiFiInterface found!");

features/lwipstack/.mbedignore

@@ -3,5 +3,5 @@ lwip/test/*
 lwip/src/apps/*
 lwip/src/netif/lwip_slipif.c
 lwip/src/include/lwip/apps/*
-lwip/src/include/posix/*
+lwip/src/include/compat/stdc/*
 lwip-eth/*

features/lwipstack/LWIPStack.cpp

@@ -514,19 +514,9 @@ nsapi_error_t LWIP::setsockopt(nsapi_socket_t handle, int level, int optname, co
             if (optlen > NSAPI_INTERFACE_NAME_MAX_SIZE) {
                 return NSAPI_ERROR_UNSUPPORTED;
             }
-#if LWIP_TCP
-            if (NETCONNTYPE_GROUP(s->conn->type) == NETCONN_TCP) {
-                s->conn->pcb.tcp->interface_name = (const char *)optval;
-            }
-#endif
-            if (NETCONNTYPE_GROUP(s->conn->type) == NETCONN_UDP) {
-                s->conn->pcb.udp->interface_name = (const char *)optval;
-            }
-#if LWIP_RAW
-            if (NETCONNTYPE_GROUP(s->conn->type) == NETCONN_RAW) {
-                s->conn->pcb.raw->interface_name = (const char *)optval;
-            }
-#endif
+
+            netconn_bind_if(s->conn,  netif_name_to_index((const char *)optval));
+
             return 0;
 #if LWIP_TCP
         case NSAPI_KEEPALIVE:

features/lwipstack/lwip/src/core/ipv4/lwip_dhcp.c

@@ -673,7 +673,7 @@ dhcp_handle_ack(struct netif *netif, struct dhcp_msg *msg_in)
   for (n = 0; (n < LWIP_DHCP_PROVIDE_DNS_SERVERS) && dhcp_option_given(dhcp, DHCP_OPTION_IDX_DNS_SERVER + n); n++) {
     ip_addr_t dns_addr;
     ip_addr_set_ip4_u32_val(dns_addr, lwip_htonl(dhcp_get_option_value(dhcp, DHCP_OPTION_IDX_DNS_SERVER + n)));
-    dns_setserver(n, &dns_addr);
+    dns_setserver(n, &dns_addr,netif);
   }
 #endif /* LWIP_DHCP_PROVIDE_DNS_SERVERS */
 }

features/lwipstack/lwip/src/core/ipv6/lwip_ip6.c

@@ -96,8 +96,8 @@ ip6_route(const ip6_addr_t *src, const ip6_addr_t *dest)
 
   /* If single netif configuration, fast return. */
   if ((netif_list != NULL) && (netif_list->next == NULL)) {
-    if (!netif_is_up(netif_list) || !netif_is_link_up(netif_list)){//||
-     //   (ip6_addr_has_zone(dest) && !ip6_addr_test_zone(dest, netif_list))) {
+    if (!netif_is_up(netif_list) || !netif_is_link_up(netif_list) ||
+        (ip6_addr_has_zone(dest) && !ip6_addr_test_zone(dest, netif_list))) {
       return NULL;
     }
     return netif_list;

features/lwipstack/lwip/src/core/ipv6/lwip_nd6.c

@@ -776,14 +776,14 @@ nd6_input(struct pbuf *p, struct netif *inp)
 
             if (htonl(rdnss_opt->lifetime) > 0) {
               /* TODO implement Lifetime > 0 */
-              dns_setserver(rdnss_server_idx++, &rdnss_address);
+              dns_setserver(rdnss_server_idx++, &rdnss_address, inp);
             } else {
               /* TODO implement DNS removal in dns.c */
               u8_t s;
               for (s = 0; s < DNS_MAX_SERVERS; s++) {
-                const ip_addr_t *addr = dns_getserver(s);
+                const ip_addr_t *addr = dns_getserver(s,netif_get_name(inp));
                 if(ip_addr_cmp(addr, &rdnss_address)) {
-                  dns_setserver(s, NULL);
+                  dns_setserver(s, NULL, inp);
                 }
               }
             }

features/lwipstack/lwip/src/core/lwip_dns.c

@@ -302,6 +302,7 @@ static struct dns_table_entry dns_table[DNS_TABLE_SIZE];
 static struct dns_req_entry   dns_requests[DNS_MAX_REQUESTS];
 #endif
 static ip_addr_t              dns_servers[DNS_MAX_SERVERS];
+struct dns_server_interface   *multihoming_dns_servers;
 
 #if LWIP_IPV4
 const ip_addr_t dns_mquery_v4group = DNS_MQUERY_IPV4_GROUP_INIT;
@@ -321,7 +322,7 @@ dns_init(void)
   /* initialize default DNS server address */
   ip_addr_t dnsserver;
   DNS_SERVER_ADDRESS(&dnsserver);
-  dns_setserver(0, &dnsserver);
+  dns_setserver(0, &dnsserver, NULL);
 #endif /* DNS_SERVER_ADDRESS */
 
 #if LWIP_FULL_DNS
@@ -363,15 +364,114 @@ dns_init(void)
  * @param dnsserver IP address of the DNS server to set
  */
 void
-dns_setserver(u8_t numdns, const ip_addr_t *dnsserver)
+dns_setserver(u8_t numdns, const ip_addr_t *dnsserver, struct netif *netif)
 {
-  if (numdns < DNS_MAX_SERVERS) {
-    if (dnsserver != NULL) {
-      dns_servers[numdns] = (*dnsserver);
+
+  if (netif == NULL || netif_check_default(netif)) {
+    if (numdns < DNS_MAX_SERVERS) {
+      if (dnsserver != NULL) {
+        dns_servers[numdns] = (*dnsserver);
+      } else {
+        dns_servers[numdns] = *IP_ADDR_ANY;
+      }
+    }
+  } else {
+    char name[INTERFACE_NAME_MAX_SIZE];
+    sprintf(name, "%c%c%d", netif->name[0], netif->name[1], netif->num);
+    dns_add_interface_server(numdns, name, dnsserver);
+  }
+}
+
+/**
+ * @ingroup dns
+ * Obtain one of the currently configured DNS server.
+ *
+ * @param numdns the index of the DNS server
+ * @return IP address of the indexed DNS server or "ip_addr_any" if the DNS
+ *         server has not been configured.
+ */
+const ip_addr_t*
+dns_getserver(u8_t numdns, const char *interface_name)
+{
+  if (interface_name == NULL) {
+    if (numdns < DNS_MAX_SERVERS) {
+      return &dns_servers[numdns];
     } else {
-      dns_servers[numdns] = *IP_ADDR_ANY;
+      return IP_ADDR_ANY;
+    }
+  } else {
+    return dns_get_interface_server(numdns, interface_name);
+  }
+}
+
+/**
+ * @ingroup dns
+ * Initialize one of the DNS servers.
+ *
+ * @param numdns the index of the DNS server to set must be < DNS_MAX_SERVERS
+ * @param dnsserver IP address of the DNS server to set
+ */
+void
+dns_add_interface_server(u8_t numdns, const char *interface_name, const ip_addr_t *dnsserver)
+{
+  struct dns_server_interface *new_interface_server;
+
+  if (numdns >= DNS_MAX_SERVERS) {
+    return;
+  }
+
+  if (multihoming_dns_servers != NULL) {
+    // if interface server already exists on the list just update it
+    for (new_interface_server = multihoming_dns_servers; new_interface_server != NULL; new_interface_server = new_interface_server->next) {
+      if (!strcmp(interface_name, new_interface_server->interface_name)) {
+        new_interface_server->dns_servers[numdns] = (*dnsserver);
+        return;
+      }
     }
   }
+  // add new dns server to the list tail
+  new_interface_server = mem_malloc(sizeof(struct dns_server_interface));
+  strncpy(new_interface_server->interface_name, interface_name, INTERFACE_NAME_MAX_SIZE);
+  new_interface_server->dns_servers[numdns] = (*dnsserver);
+  new_interface_server->next = NULL;
+
+  if (multihoming_dns_servers == NULL) {
+    multihoming_dns_servers = new_interface_server;
+  } else {
+    struct dns_server_interface *tail;
+    tail = multihoming_dns_servers;
+
+    while (tail->next != NULL) {
+      tail = tail->next;
+    }
+    tail->next = new_interface_server;
+
+  }
+}
+
+void
+dns_remove_interface_servers(const char *interface_name)
+{
+  struct dns_server_interface *temp = multihoming_dns_servers;
+  struct dns_server_interface *prev = NULL;
+
+  if (temp != NULL && !strcmp(interface_name, temp->interface_name)) {
+    multihoming_dns_servers = temp->next;
+    mem_free(temp);
+    return;
+  }
+
+  while (temp != NULL && strcmp(interface_name, temp->interface_name)) {
+    prev = temp;
+    temp = temp->next;
+  }
+
+  if (temp == NULL) {
+    return;
+  }
+
+  prev->next = temp->next;
+  mem_free(temp);
 }
 
 /**
@@ -383,13 +483,20 @@ dns_setserver(u8_t numdns, const ip_addr_t *dnsserver)
  *         server has not been configured.
  */
 const ip_addr_t *
-dns_getserver(u8_t numdns)
+dns_get_interface_server(u8_t numdns, const char *interface_name)
 {
+  struct dns_server_interface *interface_server;
+
   if (numdns < DNS_MAX_SERVERS) {
-    return &dns_servers[numdns];
-  } else {
     return IP_ADDR_ANY;
   }
+
+  for (interface_server = multihoming_dns_servers; interface_server != NULL; interface_server = interface_server->next) {
+    if (!strcmp(interface_name, interface_server->interface_name)) {
+      return &interface_server->dns_servers[numdns];
+    }
+  }
+  return IP_ADDR_ANY;
 }
 
 /**

features/lwipstack/lwip/src/core/lwip_netif.c

@@ -113,6 +113,7 @@ struct netif *netif_default;
 
 #define netif_index_to_num(index)   ((index) - 1)
 static u8_t netif_num;
+static char netif_name [INTERFACE_NAME_MAX_SIZE];
 
 #if LWIP_NUM_NETIF_CLIENT_DATA > 0
 static u8_t netif_client_id;
@@ -835,6 +836,26 @@ netif_set_default(struct netif *netif)
                             netif ? netif->name[0] : '\'', netif ? netif->name[1] : '\''));
 }
 
+/**
+ * @ingroup netif
+ * Checks if  network interface is the default network interface
+ * @param netif the default network interface
+ * @return true if netif is  set to default one
+ * otherwise return false
+ */
+bool
+netif_check_default(struct netif *netif)
+{
+  if (netif == NULL) {
+    return false;
+  }
+  if (netif_default == netif) {
+    return true;
+  } else {
+    return false;
+  }
+}
+
 /**
  * @ingroup netif
  * Bring an interface up, available for processing
@@ -1722,6 +1743,23 @@ netif_find(const char *name)
   LWIP_DEBUGF(NETIF_DEBUG, ("netif_find: didn't find %c%c\n", name[0], name[1]));
   return NULL;
 }
+ 
+/**
+ * @ingroup netif
+ * Get a network interface name
+ * @param  netif
+ * @return name the name of the netif (like netif->name) plus concatenated number
+ * in ascii representation (e.g. 'en0')
+ */
+const char *
+netif_get_name(struct netif *netif)
+{
+  if (netif == NULL) {
+    return NULL;
+  }
+  sprintf(netif_name, "%c%c%d", netif->name[0], netif->name[1], netif->num);
+  return netif_name;
+}
 
 #if LWIP_NETIF_EXT_STATUS_CALLBACK
 /**

features/lwipstack/lwip/src/core/lwip_tcp.c

@@ -1723,7 +1723,7 @@ tcp_kill_prio(u8_t prio)
    * We want to find the connections with the lowest possible prio, and among
    * these the one with the longest inactivity time.
    */
-  //mprio--; temporary commented
+   mprio--;
 
   inactivity = 0;
   inactive = NULL;