LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* BUG? behaviour mismatch between ipv4 and ipv6 in UDP rx path
@ 2011-02-07 18:29 Chris Friesen
0 siblings, 0 replies; only message in thread
From: Chris Friesen @ 2011-02-07 18:29 UTC (permalink / raw)
To: netdev, Linux Kernel Mailing List, pekkas, davem, yoshfuji
One of our guys is seeing occasional dropped ipv4 packets coming in on
an ipv6 udp socket obtained via socket(AF_INET6, SOCK_DGRAM, IPPROTO_UDP).
Here's what he says:
"The problem happens when release_sock() goes down an interesting code
path. If (sk->sk_backlog.tail) is non-NULL then release_sock() invokes
__release_sock() which loops over all queue packets and invokes the
socket's backlog receive function for each previously queued packet.
Now for the interesting part. The UDPv6 backlog receive function (in
net/ipv6/udp.c, udpv6_queue_rcv_skb()) invokes xfrm6_policy_check() to
confirm that the packet is allowed, but the problem is that it calls
this function regardless of whether the packet is IPv4 or IPv6. The
xfrm6_policy_check() function then assumes that it is an IPv6 packet and
tries to match a policy based on its packet header... but that clearly
won't work because the addresses that it finds when it decodes the skb
are completely bogus."
Looking at the ipv4 code, git commit 9382177 split __udp_queue_rcv_skb()
out of udp_queue_rcv_skb(). It was done for locking purposes, but it
also means that backlog_rcv is bound to __udp_queue_rcv_skb(), which
doesn't call xfrm4_policy_check().
Should a new function __udpv6_queue_rcv_skb() be split out from
udpv6_queue_rcv_skb() and bound to backlog_rcv to resolve the xfrm
issue? What about the locking that was the reason for the split in the
ipv4 case--is there a similar problem with ipv6?
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2011-02-07 18:31 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-02-07 18:29 BUG? behaviour mismatch between ipv4 and ipv6 in UDP rx path Chris Friesen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).