From 2c2b5e6738a4e91795343c1339eafbe83e7a24bd Mon Sep 17 00:00:00 2001 From: Gregory Nutt Date: Fri, 21 Nov 2014 17:44:12 -0600 Subject: [PATCH] Fix routing of muli-network UDP packets --- net/udp/udp_conn.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/net/udp/udp_conn.c b/net/udp/udp_conn.c index e45abc540d..42c4cf6c13 100644 --- a/net/udp/udp_conn.c +++ b/net/udp/udp_conn.c @@ -350,6 +350,10 @@ FAR struct udp_conn_s *udp_active(FAR struct udp_iphdr_s *buf) * number in the received packet. * - The remote port number is checked if the connection is bound * to a remote port. + * - If multiple network interfaces are supported, then the local + * IP address is available and we will insist that the + * destination IP matches the bound address (or the destination + * IP address is a broadcase address). * - Finally, if the connection is bound to a remote IP address, * the source IP address of the packet is checked. Broadcast * addresses are also accepted. @@ -360,6 +364,10 @@ FAR struct udp_conn_s *udp_active(FAR struct udp_iphdr_s *buf) if (conn->lport != 0 && buf->destport == conn->lport && (conn->rport == 0 || buf->srcport == conn->rport) && +#ifdef CONFIG_NETDEV_MULTINIC + (net_ipaddr_hdrcmp(buf->destipaddr, &g_alloneaddr) || + net_ipaddr_hdrcmp(buf->destipaddr, &conn->lipaddr)) && +#endif (net_ipaddr_cmp(conn->ripaddr, g_allzeroaddr) || net_ipaddr_cmp(conn->ripaddr, g_alloneaddr) || net_ipaddr_hdrcmp(buf->srcipaddr, &conn->ripaddr)))