| To: | pizza@xxxxxxxxxxxx (Stuffed Crust) |
|---|---|
| Subject: | Re: [PATCH] fix long-standing bug in 2.6/2.4 skb_copy/skb_copy_expand |
| From: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
| Date: | Mon, 09 May 2005 13:04:34 +1000 |
| Cc: | davem@xxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx |
| In-reply-to: | <20050508143259.GA30676@shaftnet.org> |
| Organization: | Core |
| Sender: | netdev-bounce@xxxxxxxxxxx |
| User-agent: | tin/1.7.4-20040225 ("Benbecula") (UNIX) (Linux/2.4.27-hx-1-686-smp (i686)) |
Stuffed Crust <pizza@xxxxxxxxxxxx> wrote: > > This is, fortunately, generally true. But if the alloc_skb function > allocates extra head room (ie calls skb_reserve() on the skb before it > passes it to the callee, this doesn't quite work. Instead, it should be > rewritten as: As far as I know the alloc_skb funciton in the kernel tree doesn't do that so your patch is not necessary unless we decide to change the way alloc_skb works. If that's what you want then please provide a patch to alloc_skb and a rationale as to why we should do that. Cheers, -- Visit Openswan at http://www.openswan.org/ Email: Herbert Xu ~{PmV>HI~} <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [7/7] [IPSEC] Add XFRMA_SA/XFRMA_POLICY for delete notification, jamal |
|---|---|
| Next by Date: | Re: [2.6 patch] net/sctp/: make two functions static, Sridhar Samudrala |
| Previous by Thread: | Re: primary and secondary ip addresses, Hasso Tepper |
| Next by Thread: | Re: [PATCH] fix long-standing bug in 2.6/2.4 skb_copy/skb_copy_expand, Stuffed Crust |
| Indexes: | [Date] [Thread] [Top] [All Lists] |