[PATCH net-next] ipv6: fix RFC5722 comment

Subsystems: networking [general], networking [ipv4/ipv6], the rest

STALE5325d

3 messages, 3 authors, 2012-01-30 · open the first message on its own page

[PATCH net-next] ipv6: fix RFC5722 comment

From: Eric Dumazet <hidden>
Date: 2012-01-30 14:29:28

RFC5722 Section 4 was amended by Errata 3089

Our implementation did the right thing anyway...

Signed-off-by: Eric Dumazet <redacted>
Cc: Nicolas Dichtel <redacted>
---
 net/ipv6/reassembly.c |    7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/net/ipv6/reassembly.c b/net/ipv6/reassembly.c
index b69fae7..9447bd6 100644
--- a/net/ipv6/reassembly.c
+++ b/net/ipv6/reassembly.c
@@ -336,12 +336,11 @@ static int ip6_frag_queue(struct frag_queue *fq, struct sk_buff *skb,
 	}
 
 found:
-	/* RFC5722, Section 4:
-	 *                                  When reassembling an IPv6 datagram, if
+	/* RFC5722, Section 4, amended by Errata ID : 3089
+	 *                          When reassembling an IPv6 datagram, if
 	 *   one or more its constituent fragments is determined to be an
 	 *   overlapping fragment, the entire datagram (and any constituent
-	 *   fragments, including those not yet received) MUST be silently
-	 *   discarded.
+	 *   fragments) MUST be silently discarded.
 	 */
 
 	/* Check for overlap with preceding fragment. */

Re: [PATCH net-next] ipv6: fix RFC5722 comment

From: Nicolas Dichtel <hidden>
Date: 2012-01-30 14:42:28

Le 30/01/2012 15:29, Eric Dumazet a écrit :
RFC5722 Section 4 was amended by Errata 3089

Our implementation did the right thing anyway...

Signed-off-by: Eric Dumazet<redacted>
Cc: Nicolas Dichtel<redacted>
Acked-by: Nicolas Dichtel <redacted>
quoted hunk
---
  net/ipv6/reassembly.c |    7 +++----
  1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/net/ipv6/reassembly.c b/net/ipv6/reassembly.c
index b69fae7..9447bd6 100644
--- a/net/ipv6/reassembly.c
+++ b/net/ipv6/reassembly.c
@@ -336,12 +336,11 @@ static int ip6_frag_queue(struct frag_queue *fq, struct sk_buff *skb,
  	}

  found:
-	/* RFC5722, Section 4:
-	 *                                  When reassembling an IPv6 datagram, if
+	/* RFC5722, Section 4, amended by Errata ID : 3089
+	 *                          When reassembling an IPv6 datagram, if
Identation was an exact copy of the original rfc (not changed by the errata), 
but it's a detail ;-)


Regards,
Nicolas
  	 *   one or more its constituent fragments is determined to be an
  	 *   overlapping fragment, the entire datagram (and any constituent
-	 *   fragments, including those not yet received) MUST be silently
-	 *   discarded.
+	 *   fragments) MUST be silently discarded.
  	 */

  	/* Check for overlap with preceding fragment. */

Re: [PATCH net-next] ipv6: fix RFC5722 comment

From: David Miller <davem@davemloft.net>
Date: 2012-01-30 17:59:12

From: Eric Dumazet <redacted>
Date: Mon, 30 Jan 2012 15:29:24 +0100
RFC5722 Section 4 was amended by Errata 3089

Our implementation did the right thing anyway...

Signed-off-by: Eric Dumazet <redacted>
Cc: Nicolas Dichtel <redacted>
Applied, thanks Eric.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help