Shan Wei wrote: > @@ -349,17 +378,20 @@ static int nf_ct_frag6_queue(struct nf_ct_frag6_queue *fq, struct sk_buff *skb, > else > fq->q.fragments = skb; > > - skb->dev = NULL; > fq->q.stamp = skb->tstamp; > fq->q.meat += skb->len; > atomic_add(skb->truesize, &nf_init_frags.mem); > > /* The first fragment. > * nhoffset is obtained from the first fragment, of course. > + * Reserve dev for sending an ICMP "Fragment Reassembly Timeout" > + * message. > */ > if (offset == 0) { > fq->nhoffset = nhoff; > fq->q.last_in |= INET_FRAG_FIRST_IN; > + } else { > + skb->dev = NULL; > } We need to store the iif and perform a lookup later just as in IPv4 because the device is not reference counted and might disappear while the fragments are queued. Besides this, the patch looks good. -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html