diff options
author | Eric Dumazet <dada1@cosmosbay.com> | 2008-11-24 17:05:22 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-11-24 17:05:22 -0500 |
commit | 3755810cebf7f9902b05c7ba9baeac0fc44881be (patch) | |
tree | e4bee0e33103781034fdf6144898a67b21778ad7 | |
parent | 920de804bca61f88643bc9171bcd06f1a56c6258 (diff) |
net: Make sure BHs are disabled in sock_prot_inuse_add()
There is still a call to sock_prot_inuse_add() in af_netlink
while in a preemptable section. Add explicit BH disable around
this call.
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | net/netlink/af_netlink.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/net/netlink/af_netlink.c b/net/netlink/af_netlink.c index c7d76579bf3c..9eb895c7a2a9 100644 --- a/net/netlink/af_netlink.c +++ b/net/netlink/af_netlink.c | |||
@@ -515,7 +515,9 @@ static int netlink_release(struct socket *sock) | |||
515 | kfree(nlk->groups); | 515 | kfree(nlk->groups); |
516 | nlk->groups = NULL; | 516 | nlk->groups = NULL; |
517 | 517 | ||
518 | local_bh_disable(); | ||
518 | sock_prot_inuse_add(sock_net(sk), &netlink_proto, -1); | 519 | sock_prot_inuse_add(sock_net(sk), &netlink_proto, -1); |
520 | local_bh_enable(); | ||
519 | sock_put(sk); | 521 | sock_put(sk); |
520 | return 0; | 522 | return 0; |
521 | } | 523 | } |