diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2011-03-30 16:57:46 -0700 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-03-30 16:57:46 -0700 |
commit | e2666f84958adb3a034b98e99699b55705117e01 (patch) | |
tree | f8fdca8163bf15d98468c63d4d98932ded0ec96f /net | |
parent | c031235b395433350f25943b7580a5e343c7b7b2 (diff) | |
download | lwn-e2666f84958adb3a034b98e99699b55705117e01.tar.gz lwn-e2666f84958adb3a034b98e99699b55705117e01.zip |
fib: add rtnl locking in ip_fib_net_exit
Daniel J Blueman reported a lockdep splat in trie_firstleaf(), caused by
RTNL being not locked before a call to fib_table_flush()
Reported-by: Daniel J Blueman <daniel.blueman@gmail.com>
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r-- | net/ipv4/fib_frontend.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/ipv4/fib_frontend.c b/net/ipv4/fib_frontend.c index f116ce8f1b46..451088330bbb 100644 --- a/net/ipv4/fib_frontend.c +++ b/net/ipv4/fib_frontend.c @@ -1068,6 +1068,7 @@ static void ip_fib_net_exit(struct net *net) fib4_rules_exit(net); #endif + rtnl_lock(); for (i = 0; i < FIB_TABLE_HASHSZ; i++) { struct fib_table *tb; struct hlist_head *head; @@ -1080,6 +1081,7 @@ static void ip_fib_net_exit(struct net *net) fib_free_table(tb); } } + rtnl_unlock(); kfree(net->ipv4.fib_table_hash); } |