osmith submitted this change.

View Change


Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved pespin: Looks good to me, but someone else must approve
gtp-link: set IPv6 socket only

GTP driver bails out for IPv4-mapped-IPv6 socket with EADDRNOAVAIL,
to prevent issues with setsockopt IPV6_ADDRFORM.

GTP control plane checks that tunnel family matches the socket family
for this GTP device, ie. there is a 1:1 mapping between the socket
listener and the device which determines the supported IP tunnel header.

Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Change-Id: I887a107657059adeb14ae425576ae7ea9018f762
---
M tools/gtp-link.c
1 file changed, 22 insertions(+), 0 deletions(-)

diff --git a/tools/gtp-link.c b/tools/gtp-link.c
index feb2efe..325a45f 100644
--- a/tools/gtp-link.c
+++ b/tools/gtp-link.c
@@ -75,6 +75,7 @@
{
int fd1 = socket(family, SOCK_DGRAM, 0);
int fd2 = socket(family, SOCK_DGRAM, 0);
+ int one = 1;

if (fd1 < 0 || fd2 < 0)
return -1;
@@ -95,6 +96,10 @@
gtp_sock->len = sizeof(struct sockaddr_in6);
setup_sockaddr_in6(&gtp_sock->sockaddr.fd1.in6, 3386);
setup_sockaddr_in6(&gtp_sock->sockaddr.fd2.in6, 2152);
+ if (setsockopt(fd1, IPPROTO_IPV6, IPV6_V6ONLY, &one, sizeof(one)) < 0)
+ perror("setsockopt IPV6_V6ONLY: ");
+ if (setsockopt(fd2, IPPROTO_IPV6, IPV6_V6ONLY, &one, sizeof(one)) < 0)
+ perror("setsockopt IPV6_V6ONLY: ");
break;
}


To view, visit change 35986. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: libgtpnl
Gerrit-Branch: master
Gerrit-Change-Id: I887a107657059adeb14ae425576ae7ea9018f762
Gerrit-Change-Number: 35986
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: osmith <osmith@sysmocom.de>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>
Gerrit-MessageType: merged