| To: | Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: Fwd: [2.6] ethertap and af_inet.c assertion failures |
| From: | "David S. Miller" <davem@xxxxxxxxxxxxx> |
| Date: | Mon, 17 Jan 2005 13:37:51 -0800 |
| Cc: | tommy.christensen@xxxxxxxxx, simon.roscic@xxxxxxxxx, netdev@xxxxxxxxxxx |
| In-reply-to: | <E1Cpo1q-0007lI-00@xxxxxxxxxxxxxxxxxxxxxxxx> |
| References: | <41E44476.20303@xxxxxxxxx> <E1Cpo1q-0007lI-00@xxxxxxxxxxxxxxxxxxxxxxxx> |
| Sender: | netdev-bounce@xxxxxxxxxxx |
On Sun, 16 Jan 2005 00:31:46 +1100 Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> wrote: > Tommy Christensen <tommy.christensen@xxxxxxxxx> wrote: > > > > netlink has messed up the send buffer accounting, when trimming a skb. > > You are spot on. Dave, you were right too about checking for charged > skb's in netlink_trim. It's good to know that I'm legitimately paranoid from time to time :-) > Since the skb will be orphaned on its way to the destination anyway, > let's simply do that before the trimming. This shouldn't lead to > skb leakage since the skb will either be charged to the destination > socket or freed. > > Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Looks good, I'll apply this. Thanks Herbert. |
| <Prev in Thread] | Current Thread | [Next in Thread> |
|---|---|---|
| ||
| Previous by Date: | Re: [NETLINK] Do not netlink_unicast shared packet in kernel/audit.c, David S. Miller |
|---|---|
| Next by Date: | Re: Iproute2: update, jamal |
| Previous by Thread: | Re: Fwd: [2.6] ethertap and af_inet.c assertion failures, Simon Roscic |
| Next by Thread: | [PATCH PKT_SCHED 0/6]: assorted fixes, Patrick McHardy |
| Indexes: | [Date] [Thread] [Top] [All Lists] |