mirror of
				https://github.com/torvalds/linux.git
				synced 2025-10-31 16:48:26 +02:00 
			
		
		
		
	ipv6: set skb->protocol on tcp, raw and ip6_append_data genereated skbs
Currently we don't initialize skb->protocol when transmitting data via tcp, raw(with and without inclhdr) or udp+ufo or appending data directly to the socket transmit queue (via ip6_append_data). This needs to be done so that we can get the correct mtu in the xfrm layer. Setting of skb->protocol happens only in functions where we also have a transmitting socket and a new skb, so we don't overwrite old values. Cc: Steffen Klassert <steffen.klassert@secunet.com> Cc: Eric Dumazet <eric.dumazet@gmail.com> Acked-by: Eric Dumazet <edumazet@google.com> Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org> Signed-off-by: Steffen Klassert <steffen.klassert@secunet.com>
This commit is contained in:
		
							parent
							
								
									5a25cf1e31
								
							
						
					
					
						commit
						9c9c9ad5fa
					
				
					 2 changed files with 4 additions and 0 deletions
				
			
		|  | @ -238,6 +238,7 @@ int ip6_xmit(struct sock *sk, struct sk_buff *skb, struct flowi6 *fl6, | ||||||
| 	hdr->saddr = fl6->saddr; | 	hdr->saddr = fl6->saddr; | ||||||
| 	hdr->daddr = *first_hop; | 	hdr->daddr = *first_hop; | ||||||
| 
 | 
 | ||||||
|  | 	skb->protocol = htons(ETH_P_IPV6); | ||||||
| 	skb->priority = sk->sk_priority; | 	skb->priority = sk->sk_priority; | ||||||
| 	skb->mark = sk->sk_mark; | 	skb->mark = sk->sk_mark; | ||||||
| 
 | 
 | ||||||
|  | @ -1057,6 +1058,7 @@ static inline int ip6_ufo_append_data(struct sock *sk, | ||||||
| 		/* initialize protocol header pointer */ | 		/* initialize protocol header pointer */ | ||||||
| 		skb->transport_header = skb->network_header + fragheaderlen; | 		skb->transport_header = skb->network_header + fragheaderlen; | ||||||
| 
 | 
 | ||||||
|  | 		skb->protocol = htons(ETH_P_IPV6); | ||||||
| 		skb->ip_summed = CHECKSUM_PARTIAL; | 		skb->ip_summed = CHECKSUM_PARTIAL; | ||||||
| 		skb->csum = 0; | 		skb->csum = 0; | ||||||
| 	} | 	} | ||||||
|  | @ -1359,6 +1361,7 @@ int ip6_append_data(struct sock *sk, int getfrag(void *from, char *to, | ||||||
| 			/*
 | 			/*
 | ||||||
| 			 *	Fill in the control structures | 			 *	Fill in the control structures | ||||||
| 			 */ | 			 */ | ||||||
|  | 			skb->protocol = htons(ETH_P_IPV6); | ||||||
| 			skb->ip_summed = CHECKSUM_NONE; | 			skb->ip_summed = CHECKSUM_NONE; | ||||||
| 			skb->csum = 0; | 			skb->csum = 0; | ||||||
| 			/* reserve for fragmentation and ipsec header */ | 			/* reserve for fragmentation and ipsec header */ | ||||||
|  |  | ||||||
|  | @ -628,6 +628,7 @@ static int rawv6_send_hdrinc(struct sock *sk, void *from, int length, | ||||||
| 		goto error; | 		goto error; | ||||||
| 	skb_reserve(skb, hlen); | 	skb_reserve(skb, hlen); | ||||||
| 
 | 
 | ||||||
|  | 	skb->protocol = htons(ETH_P_IPV6); | ||||||
| 	skb->priority = sk->sk_priority; | 	skb->priority = sk->sk_priority; | ||||||
| 	skb->mark = sk->sk_mark; | 	skb->mark = sk->sk_mark; | ||||||
| 	skb_dst_set(skb, &rt->dst); | 	skb_dst_set(skb, &rt->dst); | ||||||
|  |  | ||||||
		Loading…
	
		Reference in a new issue
	
	 Hannes Frederic Sowa
						Hannes Frederic Sowa