Re: test13-pre4 ip defrag oops

From: kuznet@ms2.inr.ac.ru
Date: Sun Dec 24 2000 - 12:27:25 EST


Hello!

> eax: 20202037 ebx: d3a406c0 ecx: cf683024 edx: c734a2a0

Ough... found eventually. skb->dev turns out to be not initialized. 8)8)

This patchlet surely fixes the bug. (plus writes are ordered)

Alexey

--- ../vger3-001222/linux/net/core/skbuff.c Fri Dec 22 19:37:54 2000
+++ linux/net/core/skbuff.c Sun Dec 24 20:24:20 2000
@@ -227,15 +227,20 @@
 {
         struct sk_buff *skb = p;
 
- skb->destructor = NULL;
- skb->pkt_type = PACKET_HOST; /* Default type */
- skb->prev = skb->next = NULL;
+ skb->next = NULL;
+ skb->prev = NULL;
         skb->list = NULL;
         skb->sk = NULL;
         skb->stamp.tv_sec=0; /* No idea about time */
+ skb->dev = NULL;
+ skb->dst = NULL;
+ memset(skb->cb, 0, sizeof(skb->cb));
+ skb->pkt_type = PACKET_HOST; /* Default type */
         skb->ip_summed = 0;
+ skb->priority = 0;
         skb->security = 0; /* By default packets are insecure */
- skb->dst = NULL;
+ skb->destructor = NULL;
+
 #ifdef CONFIG_NETFILTER
         skb->nfmark = skb->nfcache = 0;
         skb->nfct = NULL;
@@ -246,8 +251,6 @@
 #ifdef CONFIG_NET_SCHED
         skb->tc_index = 0;
 #endif
- memset(skb->cb, 0, sizeof(skb->cb));
- skb->priority = 0;
 }
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Dec 31 2000 - 21:00:07 EST