Commit 703133de authored by Ansis Atteka's avatar Ansis Atteka Committed by David S. Miller
Browse files

ip: generate unique IP identificator if local fragmentation is allowed 



If local fragmentation is allowed, then ip_select_ident() and
ip_select_ident_more() need to generate unique IDs to ensure
correct defragmentation on the peer.

For example, if IPsec (tunnel mode) has to encrypt large skbs
that have local_df bit set, then all IP fragments that belonged
to different ESP datagrams would have used the same identificator.
If one of these IP fragments would get lost or reordered, then
peer could possibly stitch together wrong IP fragments that did
not belong to the same datagram. This would lead to a packet loss
or data corruption.

Signed-off-by: default avatarAnsis Atteka <aatteka@nicira.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 749154aa

drivers/net/ppp/pptp.c

+1 −1
Original line number Diff line number Diff line
@@ -281,7 +281,7 @@ static int pptp_xmit(struct ppp_channel *chan, struct sk_buff *skb) 
	nf_reset(skb);



	skb->ip_summed = CHECKSUM_NONE;

	ip_select_ident(iph, &rt->dst, NULL);

	ip_select_ident(skb, &rt->dst, NULL);

	ip_send_check(iph);



	ip_local_out(skb);

include/net/ip.h

+8 −4
Original line number Diff line number Diff line
@@ -264,9 +264,11 @@ int ip_dont_fragment(struct sock *sk, struct dst_entry *dst) 


extern void __ip_select_ident(struct iphdr *iph, struct dst_entry *dst, int more);



static inline void ip_select_ident(struct iphdr *iph, struct dst_entry *dst, struct sock *sk)

static inline void ip_select_ident(struct sk_buff *skb, struct dst_entry *dst, struct sock *sk)

{

	if (iph->frag_off & htons(IP_DF)) {

	struct iphdr *iph = ip_hdr(skb);



	if ((iph->frag_off & htons(IP_DF)) && !skb->local_df) {

		/* This is only to work around buggy Windows95/2000

		 * VJ compression implementations.  If the ID field

		 * does not change, they drop every other packet in
@@ -278,9 +280,11 @@ static inline void ip_select_ident(struct iphdr *iph, struct dst_entry *dst, str 
		__ip_select_ident(iph, dst, 0);

}



static inline void ip_select_ident_more(struct iphdr *iph, struct dst_entry *dst, struct sock *sk, int more)

static inline void ip_select_ident_more(struct sk_buff *skb, struct dst_entry *dst, struct sock *sk, int more)

{

	if (iph->frag_off & htons(IP_DF)) {

	struct iphdr *iph = ip_hdr(skb);



	if ((iph->frag_off & htons(IP_DF)) && !skb->local_df) {

		if (sk && inet_sk(sk)->inet_daddr) {

			iph->id = htons(inet_sk(sk)->inet_id);

			inet_sk(sk)->inet_id += 1 + more;

net/ipv4/igmp.c

+2 −2
Original line number Diff line number Diff line
@@ -369,7 +369,7 @@ static struct sk_buff *igmpv3_newpack(struct net_device *dev, int size) 
	pip->saddr    = fl4.saddr;

	pip->protocol = IPPROTO_IGMP;

	pip->tot_len  = 0;	/* filled in later */

	ip_select_ident(pip, &rt->dst, NULL);

	ip_select_ident(skb, &rt->dst, NULL);

	((u8 *)&pip[1])[0] = IPOPT_RA;

	((u8 *)&pip[1])[1] = 4;

	((u8 *)&pip[1])[2] = 0;
@@ -714,7 +714,7 @@ static int igmp_send_report(struct in_device *in_dev, struct ip_mc_list *pmc, 
	iph->daddr    = dst;

	iph->saddr    = fl4.saddr;

	iph->protocol = IPPROTO_IGMP;

	ip_select_ident(iph, &rt->dst, NULL);

	ip_select_ident(skb, &rt->dst, NULL);

	((u8 *)&iph[1])[0] = IPOPT_RA;

	((u8 *)&iph[1])[1] = 4;

	((u8 *)&iph[1])[2] = 0;

net/ipv4/inetpeer.c

+2 −2
Original line number Diff line number Diff line
@@ -32,8 +32,8 @@ 
 *  At the moment of writing this notes identifier of IP packets is generated

 *  to be unpredictable using this code only for packets subjected

 *  (actually or potentially) to defragmentation.  I.e. DF packets less than

 *  PMTU in size uses a constant ID and do not use this code (see

 *  ip_select_ident() in include/net/ip.h).

 *  PMTU in size when local fragmentation is disabled use a constant ID and do

 *  not use this code (see ip_select_ident() in include/net/ip.h).

 *

 *  Route cache entries hold references to our nodes.

 *  New cache entries get references via lookup by destination IP address in

net/ipv4/ip_output.c

+3 −3
Original line number Diff line number Diff line
@@ -148,7 +148,7 @@ int ip_build_and_send_pkt(struct sk_buff *skb, struct sock *sk, 
	iph->daddr    = (opt && opt->opt.srr ? opt->opt.faddr : daddr);

	iph->saddr    = saddr;

	iph->protocol = sk->sk_protocol;

	ip_select_ident(iph, &rt->dst, sk);

	ip_select_ident(skb, &rt->dst, sk);



	if (opt && opt->opt.optlen) {

		iph->ihl += opt->opt.optlen>>2;
@@ -386,7 +386,7 @@ packet_routed: 
		ip_options_build(skb, &inet_opt->opt, inet->inet_daddr, rt, 0);

	}



	ip_select_ident_more(iph, &rt->dst, sk,

	ip_select_ident_more(skb, &rt->dst, sk,

			     (skb_shinfo(skb)->gso_segs ?: 1) - 1);



	skb->priority = sk->sk_priority;
@@ -1324,7 +1324,7 @@ struct sk_buff *__ip_make_skb(struct sock *sk, 
	iph->ttl = ttl;

	iph->protocol = sk->sk_protocol;

	ip_copy_addrs(iph, fl4);

	ip_select_ident(iph, &rt->dst, sk);

	ip_select_ident(skb, &rt->dst, sk);



	if (opt) {

		iph->ihl += opt->optlen>>2;

CRA Git | Maintained and supported by SUSTech CRA and CCSE