forked from mirrors/linux
		
	ipv6: honor IPV6_PKTINFO with v4 mapped addresses on sendmsg
In case we decide in udp6_sendmsg to send the packet down the ipv4 udp_sendmsg path because the destination is either of family AF_INET or the destination is an ipv4 mapped ipv6 address, we don't honor the maybe specified ipv4 mapped ipv6 address in IPV6_PKTINFO. We simply can check for this option in ip_cmsg_send because no calls to ipv6 module functions are needed to do so. Reported-by: Gert Doering <gert@space.net> Cc: Tore Anderson <tore@fud.no> Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									21f374c6cf
								
							
						
					
					
						commit
						c8e6ad0829
					
				
					 5 changed files with 24 additions and 5 deletions
				
			
		| 
						 | 
					@ -489,7 +489,8 @@ int ip_options_rcv_srr(struct sk_buff *skb);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
void ipv4_pktinfo_prepare(const struct sock *sk, struct sk_buff *skb);
 | 
					void ipv4_pktinfo_prepare(const struct sock *sk, struct sk_buff *skb);
 | 
				
			||||||
void ip_cmsg_recv(struct msghdr *msg, struct sk_buff *skb);
 | 
					void ip_cmsg_recv(struct msghdr *msg, struct sk_buff *skb);
 | 
				
			||||||
int ip_cmsg_send(struct net *net, struct msghdr *msg, struct ipcm_cookie *ipc);
 | 
					int ip_cmsg_send(struct net *net, struct msghdr *msg,
 | 
				
			||||||
 | 
							 struct ipcm_cookie *ipc, bool allow_ipv6);
 | 
				
			||||||
int ip_setsockopt(struct sock *sk, int level, int optname, char __user *optval,
 | 
					int ip_setsockopt(struct sock *sk, int level, int optname, char __user *optval,
 | 
				
			||||||
		  unsigned int optlen);
 | 
							  unsigned int optlen);
 | 
				
			||||||
int ip_getsockopt(struct sock *sk, int level, int optname, char __user *optval,
 | 
					int ip_getsockopt(struct sock *sk, int level, int optname, char __user *optval,
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -186,7 +186,8 @@ void ip_cmsg_recv(struct msghdr *msg, struct sk_buff *skb)
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
EXPORT_SYMBOL(ip_cmsg_recv);
 | 
					EXPORT_SYMBOL(ip_cmsg_recv);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
int ip_cmsg_send(struct net *net, struct msghdr *msg, struct ipcm_cookie *ipc)
 | 
					int ip_cmsg_send(struct net *net, struct msghdr *msg, struct ipcm_cookie *ipc,
 | 
				
			||||||
 | 
							 bool allow_ipv6)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
	int err, val;
 | 
						int err, val;
 | 
				
			||||||
	struct cmsghdr *cmsg;
 | 
						struct cmsghdr *cmsg;
 | 
				
			||||||
| 
						 | 
					@ -194,6 +195,22 @@ int ip_cmsg_send(struct net *net, struct msghdr *msg, struct ipcm_cookie *ipc)
 | 
				
			||||||
	for (cmsg = CMSG_FIRSTHDR(msg); cmsg; cmsg = CMSG_NXTHDR(msg, cmsg)) {
 | 
						for (cmsg = CMSG_FIRSTHDR(msg); cmsg; cmsg = CMSG_NXTHDR(msg, cmsg)) {
 | 
				
			||||||
		if (!CMSG_OK(msg, cmsg))
 | 
							if (!CMSG_OK(msg, cmsg))
 | 
				
			||||||
			return -EINVAL;
 | 
								return -EINVAL;
 | 
				
			||||||
 | 
					#if defined(CONFIG_IPV6)
 | 
				
			||||||
 | 
							if (allow_ipv6 &&
 | 
				
			||||||
 | 
							    cmsg->cmsg_level == SOL_IPV6 &&
 | 
				
			||||||
 | 
							    cmsg->cmsg_type == IPV6_PKTINFO) {
 | 
				
			||||||
 | 
								struct in6_pktinfo *src_info;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
								if (cmsg->cmsg_len < CMSG_LEN(sizeof(*src_info)))
 | 
				
			||||||
 | 
									return -EINVAL;
 | 
				
			||||||
 | 
								src_info = (struct in6_pktinfo *)CMSG_DATA(cmsg);
 | 
				
			||||||
 | 
								if (!ipv6_addr_v4mapped(&src_info->ipi6_addr))
 | 
				
			||||||
 | 
									return -EINVAL;
 | 
				
			||||||
 | 
								ipc->oif = src_info->ipi6_ifindex;
 | 
				
			||||||
 | 
								ipc->addr = src_info->ipi6_addr.s6_addr32[3];
 | 
				
			||||||
 | 
								continue;
 | 
				
			||||||
 | 
							}
 | 
				
			||||||
 | 
					#endif
 | 
				
			||||||
		if (cmsg->cmsg_level != SOL_IP)
 | 
							if (cmsg->cmsg_level != SOL_IP)
 | 
				
			||||||
			continue;
 | 
								continue;
 | 
				
			||||||
		switch (cmsg->cmsg_type) {
 | 
							switch (cmsg->cmsg_type) {
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -727,7 +727,7 @@ static int ping_v4_sendmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *m
 | 
				
			||||||
	sock_tx_timestamp(sk, &ipc.tx_flags);
 | 
						sock_tx_timestamp(sk, &ipc.tx_flags);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (msg->msg_controllen) {
 | 
						if (msg->msg_controllen) {
 | 
				
			||||||
		err = ip_cmsg_send(sock_net(sk), msg, &ipc);
 | 
							err = ip_cmsg_send(sock_net(sk), msg, &ipc, false);
 | 
				
			||||||
		if (err)
 | 
							if (err)
 | 
				
			||||||
			return err;
 | 
								return err;
 | 
				
			||||||
		if (ipc.opt)
 | 
							if (ipc.opt)
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -524,7 +524,7 @@ static int raw_sendmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *msg,
 | 
				
			||||||
	ipc.oif = sk->sk_bound_dev_if;
 | 
						ipc.oif = sk->sk_bound_dev_if;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (msg->msg_controllen) {
 | 
						if (msg->msg_controllen) {
 | 
				
			||||||
		err = ip_cmsg_send(sock_net(sk), msg, &ipc);
 | 
							err = ip_cmsg_send(sock_net(sk), msg, &ipc, false);
 | 
				
			||||||
		if (err)
 | 
							if (err)
 | 
				
			||||||
			goto out;
 | 
								goto out;
 | 
				
			||||||
		if (ipc.opt)
 | 
							if (ipc.opt)
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
| 
						 | 
					@ -931,7 +931,8 @@ int udp_sendmsg(struct kiocb *iocb, struct sock *sk, struct msghdr *msg,
 | 
				
			||||||
	sock_tx_timestamp(sk, &ipc.tx_flags);
 | 
						sock_tx_timestamp(sk, &ipc.tx_flags);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (msg->msg_controllen) {
 | 
						if (msg->msg_controllen) {
 | 
				
			||||||
		err = ip_cmsg_send(sock_net(sk), msg, &ipc);
 | 
							err = ip_cmsg_send(sock_net(sk), msg, &ipc,
 | 
				
			||||||
 | 
									   sk->sk_family == AF_INET6);
 | 
				
			||||||
		if (err)
 | 
							if (err)
 | 
				
			||||||
			return err;
 | 
								return err;
 | 
				
			||||||
		if (ipc.opt)
 | 
							if (ipc.opt)
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue