]> git.baikalelectronics.ru Git - kernel.git/commit
net: avoid reference counter overflows on fib_rules in multicast forwarding
authorHannes Frederic Sowa <hannes@stressinduktion.org>
Mon, 13 Jan 2014 01:45:22 +0000 (02:45 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 15 Jan 2014 01:37:25 +0000 (17:37 -0800)
commitf4d84c95073880ca7b50873177018728c07b9729
tree04574bc5a4f4b60133b3dbc72c5174a9f847b3b4
parentffd0cff07977422fb99fa6d98a8c781422409b07
net: avoid reference counter overflows on fib_rules in multicast forwarding

Bob Falken reported that after 4G packets, multicast forwarding stopped
working. This was because of a rule reference counter overflow which
freed the rule as soon as the overflow happend.

This patch solves this by adding the FIB_LOOKUP_NOREF flag to
fib_rules_lookup calls. This is safe even from non-rcu locked sections
as in this case the flag only implies not taking a reference to the rule,
which we don't need at all.

Rules only hold references to the namespace, which are guaranteed to be
available during the call of the non-rcu protected function reg_vif_xmit
because of the interface reference which itself holds a reference to
the net namespace.

Fixes: ae9451906e7908 ("ipv4: ipmr: support multiple tables")
Fixes: fa4a490165bddf ("ipv6: ip6mr: support multiple tables")
Reported-by: Bob Falken <NetFestivalHaveFun@gmx.com>
Cc: Patrick McHardy <kaber@trash.net>
Cc: Thomas Graf <tgraf@suug.ch>
Cc: Julian Anastasov <ja@ssi.bg>
Cc: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Acked-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ipmr.c
net/ipv6/ip6mr.c