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

From
Nicolas Dichtel <hidden>
Date
2012-01-30 14:42:28
Reply to
Eric Dumazet, 2012-01-30
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 ↗ jump to 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
quoted hunk ↗ jump to hunk
  	 *   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. */
Thread (3 messages) 3 messages, 3 authors, last reply 2012-01-30
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help