forked from mirrors/linux
		
	mptcp: sockopt: add SOL_IP freebind & transparent options
These options also need to be set before bind, so do the sync of msk to new ssk socket a bit earlier. Signed-off-by: Florian Westphal <fw@strlen.de> Signed-off-by: Mat Martineau <mathew.j.martineau@linux.intel.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									ffcacff87c
								
							
						
					
					
						commit
						c9406a23c1
					
				
					 2 changed files with 67 additions and 2 deletions
				
			
		|  | @ -390,6 +390,8 @@ static int mptcp_setsockopt_v6(struct mptcp_sock *msk, int optname, | ||||||
| 
 | 
 | ||||||
| 	switch (optname) { | 	switch (optname) { | ||||||
| 	case IPV6_V6ONLY: | 	case IPV6_V6ONLY: | ||||||
|  | 	case IPV6_TRANSPARENT: | ||||||
|  | 	case IPV6_FREEBIND: | ||||||
| 		lock_sock(sk); | 		lock_sock(sk); | ||||||
| 		ssock = __mptcp_nmpc_socket(msk); | 		ssock = __mptcp_nmpc_socket(msk); | ||||||
| 		if (!ssock) { | 		if (!ssock) { | ||||||
|  | @ -398,8 +400,24 @@ static int mptcp_setsockopt_v6(struct mptcp_sock *msk, int optname, | ||||||
| 		} | 		} | ||||||
| 
 | 
 | ||||||
| 		ret = tcp_setsockopt(ssock->sk, SOL_IPV6, optname, optval, optlen); | 		ret = tcp_setsockopt(ssock->sk, SOL_IPV6, optname, optval, optlen); | ||||||
| 		if (ret == 0) | 		if (ret != 0) { | ||||||
|  | 			release_sock(sk); | ||||||
|  | 			return ret; | ||||||
|  | 		} | ||||||
|  | 
 | ||||||
|  | 		sockopt_seq_inc(msk); | ||||||
|  | 
 | ||||||
|  | 		switch (optname) { | ||||||
|  | 		case IPV6_V6ONLY: | ||||||
| 			sk->sk_ipv6only = ssock->sk->sk_ipv6only; | 			sk->sk_ipv6only = ssock->sk->sk_ipv6only; | ||||||
|  | 			break; | ||||||
|  | 		case IPV6_TRANSPARENT: | ||||||
|  | 			inet_sk(sk)->transparent = inet_sk(ssock->sk)->transparent; | ||||||
|  | 			break; | ||||||
|  | 		case IPV6_FREEBIND: | ||||||
|  | 			inet_sk(sk)->freebind = inet_sk(ssock->sk)->freebind; | ||||||
|  | 			break; | ||||||
|  | 		} | ||||||
| 
 | 
 | ||||||
| 		release_sock(sk); | 		release_sock(sk); | ||||||
| 		break; | 		break; | ||||||
|  | @ -598,6 +616,46 @@ static int mptcp_setsockopt_sol_tcp_congestion(struct mptcp_sock *msk, sockptr_t | ||||||
| 	return ret; | 	return ret; | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
|  | static int mptcp_setsockopt_sol_ip_set_transparent(struct mptcp_sock *msk, int optname, | ||||||
|  | 						   sockptr_t optval, unsigned int optlen) | ||||||
|  | { | ||||||
|  | 	struct sock *sk = (struct sock *)msk; | ||||||
|  | 	struct inet_sock *issk; | ||||||
|  | 	struct socket *ssock; | ||||||
|  | 	int err; | ||||||
|  | 
 | ||||||
|  | 	err = ip_setsockopt(sk, SOL_IP, optname, optval, optlen); | ||||||
|  | 	if (err != 0) | ||||||
|  | 		return err; | ||||||
|  | 
 | ||||||
|  | 	lock_sock(sk); | ||||||
|  | 
 | ||||||
|  | 	ssock = __mptcp_nmpc_socket(msk); | ||||||
|  | 	if (!ssock) { | ||||||
|  | 		release_sock(sk); | ||||||
|  | 		return -EINVAL; | ||||||
|  | 	} | ||||||
|  | 
 | ||||||
|  | 	issk = inet_sk(ssock->sk); | ||||||
|  | 
 | ||||||
|  | 	switch (optname) { | ||||||
|  | 	case IP_FREEBIND: | ||||||
|  | 		issk->freebind = inet_sk(sk)->freebind; | ||||||
|  | 		break; | ||||||
|  | 	case IP_TRANSPARENT: | ||||||
|  | 		issk->transparent = inet_sk(sk)->transparent; | ||||||
|  | 		break; | ||||||
|  | 	default: | ||||||
|  | 		release_sock(sk); | ||||||
|  | 		WARN_ON_ONCE(1); | ||||||
|  | 		return -EOPNOTSUPP; | ||||||
|  | 	} | ||||||
|  | 
 | ||||||
|  | 	sockopt_seq_inc(msk); | ||||||
|  | 	release_sock(sk); | ||||||
|  | 	return 0; | ||||||
|  | } | ||||||
|  | 
 | ||||||
| static int mptcp_setsockopt_v4_set_tos(struct mptcp_sock *msk, int optname, | static int mptcp_setsockopt_v4_set_tos(struct mptcp_sock *msk, int optname, | ||||||
| 				       sockptr_t optval, unsigned int optlen) | 				       sockptr_t optval, unsigned int optlen) | ||||||
| { | { | ||||||
|  | @ -627,6 +685,9 @@ static int mptcp_setsockopt_v4(struct mptcp_sock *msk, int optname, | ||||||
| 			       sockptr_t optval, unsigned int optlen) | 			       sockptr_t optval, unsigned int optlen) | ||||||
| { | { | ||||||
| 	switch (optname) { | 	switch (optname) { | ||||||
|  | 	case IP_FREEBIND: | ||||||
|  | 	case IP_TRANSPARENT: | ||||||
|  | 		return mptcp_setsockopt_sol_ip_set_transparent(msk, optname, optval, optlen); | ||||||
| 	case IP_TOS: | 	case IP_TOS: | ||||||
| 		return mptcp_setsockopt_v4_set_tos(msk, optname, optval, optlen); | 		return mptcp_setsockopt_v4_set_tos(msk, optname, optval, optlen); | ||||||
| 	} | 	} | ||||||
|  | @ -1068,6 +1129,9 @@ static void sync_socket_options(struct mptcp_sock *msk, struct sock *ssk) | ||||||
| 
 | 
 | ||||||
| 	if (inet_csk(sk)->icsk_ca_ops != inet_csk(ssk)->icsk_ca_ops) | 	if (inet_csk(sk)->icsk_ca_ops != inet_csk(ssk)->icsk_ca_ops) | ||||||
| 		tcp_set_congestion_control(ssk, msk->ca_name, false, true); | 		tcp_set_congestion_control(ssk, msk->ca_name, false, true); | ||||||
|  | 
 | ||||||
|  | 	inet_sk(ssk)->transparent = inet_sk(sk)->transparent; | ||||||
|  | 	inet_sk(ssk)->freebind = inet_sk(sk)->freebind; | ||||||
| } | } | ||||||
| 
 | 
 | ||||||
| static void __mptcp_sockopt_sync(struct mptcp_sock *msk, struct sock *ssk) | static void __mptcp_sockopt_sync(struct mptcp_sock *msk, struct sock *ssk) | ||||||
|  |  | ||||||
|  | @ -1425,6 +1425,8 @@ int __mptcp_subflow_connect(struct sock *sk, const struct mptcp_addr_info *loc, | ||||||
| 	if (addr.ss_family == AF_INET6) | 	if (addr.ss_family == AF_INET6) | ||||||
| 		addrlen = sizeof(struct sockaddr_in6); | 		addrlen = sizeof(struct sockaddr_in6); | ||||||
| #endif | #endif | ||||||
|  | 	mptcp_sockopt_sync(msk, ssk); | ||||||
|  | 
 | ||||||
| 	ssk->sk_bound_dev_if = ifindex; | 	ssk->sk_bound_dev_if = ifindex; | ||||||
| 	err = kernel_bind(sf, (struct sockaddr *)&addr, addrlen); | 	err = kernel_bind(sf, (struct sockaddr *)&addr, addrlen); | ||||||
| 	if (err) | 	if (err) | ||||||
|  | @ -1441,7 +1443,6 @@ int __mptcp_subflow_connect(struct sock *sk, const struct mptcp_addr_info *loc, | ||||||
| 	mptcp_info2sockaddr(remote, &addr, ssk->sk_family); | 	mptcp_info2sockaddr(remote, &addr, ssk->sk_family); | ||||||
| 
 | 
 | ||||||
| 	mptcp_add_pending_subflow(msk, subflow); | 	mptcp_add_pending_subflow(msk, subflow); | ||||||
| 	mptcp_sockopt_sync(msk, ssk); |  | ||||||
| 	err = kernel_connect(sf, (struct sockaddr *)&addr, addrlen, O_NONBLOCK); | 	err = kernel_connect(sf, (struct sockaddr *)&addr, addrlen, O_NONBLOCK); | ||||||
| 	if (err && err != -EINPROGRESS) | 	if (err && err != -EINPROGRESS) | ||||||
| 		goto failed_unlink; | 		goto failed_unlink; | ||||||
|  |  | ||||||
		Loading…
	
		Reference in a new issue
	
	 Florian Westphal
						Florian Westphal