diff options
| author |   Vincent Bernat <vincent@bernat.im> | 2018-07-31 21:18:11 +0200 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-08-01 09:50:04 -0700 |
| commit | 83ba4645152d1177c161750e1064e3a8e7cee19b (patch) | |
| tree | 7209c851dda98efb2aa1e7a42ff738d3fe560c38 /net/ipv4/ping.c | |
| parent | net: change Exar/Neterion menu items to be alphabetical (diff) | |
| download | linux-dev-83ba4645152d1177c161750e1064e3a8e7cee19b.tar.xz linux-dev-83ba4645152d1177c161750e1064e3a8e7cee19b.zip | |
net: add helpers checking if socket can be bound to nonlocal address
The construction "net->ipv4.sysctl_ip_nonlocal_bind || inet->freebind
|| inet->transparent" is present three times and its IPv6 counterpart
is also present three times. We introduce two small helpers to
characterize these tests uniformly.
Signed-off-by: Vincent Bernat <vincent@bernat.im>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4/ping.c')
| -rw-r--r-- | net/ipv4/ping.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/ipv4/ping.c b/net/ipv4/ping.c index b54c964ad925..8d7aaf118a30 100644 --- a/net/ipv4/ping.c +++ b/net/ipv4/ping.c @@ -320,8 +320,7 @@ static int ping_check_bind_addr(struct sock *sk, struct inet_sock *isk, if (addr->sin_addr.s_addr == htonl(INADDR_ANY)) chk_addr_ret = RTN_LOCAL; - if ((net->ipv4.sysctl_ip_nonlocal_bind == 0 && - isk->freebind == 0 && isk->transparent == 0 && + if ((!inet_can_nonlocal_bind(net, isk) && chk_addr_ret != RTN_LOCAL) || chk_addr_ret == RTN_MULTICAST || chk_addr_ret == RTN_BROADCAST) @@ -361,8 +360,7 @@ static int ping_check_bind_addr(struct sock *sk, struct inet_sock *isk, scoped); rcu_read_unlock(); - if (!(net->ipv6.sysctl.ip_nonlocal_bind || - isk->freebind || isk->transparent || has_addr || + if (!(ipv6_can_nonlocal_bind(net, isk) || has_addr || addr_type == IPV6_ADDR_ANY)) return -EADDRNOTAVAIL; |