]> git.baikalelectronics.ru Git - kernel.git/commit
[IPv6]: Fix NULL pointer dereference in ip6_flush_pending_frames
authorYOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
Tue, 11 Sep 2007 09:31:43 +0000 (11:31 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 11 Sep 2007 09:31:43 +0000 (11:31 +0200)
commit181611a27b656571730dd8b707918f770ff1ace3
tree562076978b4a8f71f4cf49903496cf95ab02d3d2
parent14b4324dbafc88dc4fef93f5b5e216ecf85ae5d7
[IPv6]: Fix NULL pointer dereference in ip6_flush_pending_frames

Some of skbs in sk->write_queue do not have skb->dst because
we do not fill skb->dst when we allocate new skb in append_data().

BTW, I think we may not need to (or we should not) increment some stats
when using corking; if 100 sendmsg() (with MSG_MORE) result in 2 packets,
how many should we increment?

If 100, we should set skb->dst for every queued skbs.

If 1 (or 2 (*)), we increment the stats for the first queued skb and
we should just skip incrementing OutDiscards for the rest of queued skbs,
adn we should also impelement this semantics in other places;
e.g., we should increment other stats just once, not 100 times.

*: depends on the place we are discarding the datagram.

I guess should just increment by 1 (or 2).

Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/ip6_output.c