Skip to content

lwip: fix socket behaviour #5684

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Dec 28, 2017
Merged

lwip: fix socket behaviour #5684

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
6d3e417
lwip: fix socket behaviour
juhaylinen Dec 11, 2017
52353d2
Add function to check if address is local
juhaylinen Dec 12, 2017
32e64e7
Fix build for IPv4+IPv6 config
juhaylinen Dec 18, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add function to check if address is local 
Update the code to check all addresses for all interfaces. Move
the code from mbed_lwip_socket_bind() to a new function called
mbed_lwip_is_local_addr()
  • Loading branch information
@juhaylinen
juhaylinen committed Dec 22, 2017
commit 52353d2ede17c6c9c40bbc724674f4557479906e
57 changes: 33 additions & 24 deletions features/FEATURE_LWIP/lwip-interface/lwip_stack.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -275,6 +275,37 @@ static const ip_addr_t *mbed_lwip_get_ipv6_addr(const struct netif *netif)
}
#endif

static bool mbed_lwip_is_local_addr(const ip_addr_t *ip_addr)
{
struct netif *netif;

for (netif = netif_list; netif != NULL; netif = netif->next) {
if (!netif_is_up(netif)) {
continue;
}
#if LWIP_IPV6
if (IP_IS_V6(ip_addr)) {
for (int i = 0; i < LWIP_IPV6_NUM_ADDRESSES; i++) {
if (ip6_addr_isvalid(netif_ip6_addr_state(netif, i)) &&
ip6_addr_cmp(netif_ip6_addr(netif, i), ip_addr)) {
return true;
}
}
}
#endif

#if LWIP_IPV4
if (IP_IS_V4(ip_addr)) {
if (!ip4_addr_isany(netif_ip4_addr(netif)) &&
ip4_addr_cmp(netif_ip_addr4(netif), ip_addr)) {
return true;
}
}
#endif
}
return false;
}

const ip_addr_t *mbed_lwip_get_ip_addr(bool any_addr, const struct netif *netif)
{
const ip_addr_t *pref_ip_addr = 0;
Expand Down Expand Up @@ -992,31 +1023,9 @@ static nsapi_error_t mbed_lwip_socket_bind(nsapi_stack_t *stack, nsapi_socket_t
return NSAPI_ERROR_PARAMETER;
}

#if LWIP_IPV6
if (IP_IS_V6(ip_addr) && !ip6_addr_isany(&ip_addr)) {
const ip_addr_t *local_addr = mbed_lwip_get_ipv6_addr(&lwip_netif);
if (!local_addr) {
return NSAPI_ERROR_PARAMETER;
}

if (!ip6_addr_cmp(local_addr, &ip_addr)) {
return NSAPI_ERROR_PARAMETER;
}
}
#endif

#if LWIP_IPV4
if (IP_IS_V4(ip_addr) && !ip4_addr_isany(&ip_addr)) {
const ip_addr_t *local_addr = mbed_lwip_get_ipv4_addr(&lwip_netif);
if (!local_addr) {
return NSAPI_ERROR_PARAMETER;
}

if (!ip4_addr_cmp(local_addr, &ip_addr)) {
return NSAPI_ERROR_PARAMETER;
}
if (!ip_addr_isany(&ip_addr) && !mbed_lwip_is_local_addr(&ip_addr)) {
return NSAPI_ERROR_PARAMETER;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if the return error here should be "NO_ADDRESS"? Guess this is fine though.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see Nanostack has "PARAMETER" already, so this matches.

}
#endif

err_t err = netconn_bind(s->conn, &ip_addr, port);
return mbed_lwip_err_remap(err);
Expand Down