mirror of
https://github.com/torvalds/linux.git
synced 2024-11-12 07:01:57 +00:00
[NETLINK]: No need for a separate __netlink_release call.
Merge it to netlink_kernel_release. Signed-off-by: Denis V. Lunev <den@openvz.org> Acked-by: Daniel Lezcano <dlezcano@fr.ibm.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
79c9115953
commit
9dfbec1fb2
@ -1344,22 +1344,6 @@ static void netlink_data_ready(struct sock *sk, int len)
|
||||
* queueing.
|
||||
*/
|
||||
|
||||
static void __netlink_release(struct sock *sk)
|
||||
{
|
||||
/*
|
||||
* Last sock_put should drop referrence to sk->sk_net. It has already
|
||||
* been dropped in netlink_kernel_create. Taking referrence to stopping
|
||||
* namespace is not an option.
|
||||
* Take referrence to a socket to remove it from netlink lookup table
|
||||
* _alive_ and after that destroy it in the context of init_net.
|
||||
*/
|
||||
|
||||
sock_hold(sk);
|
||||
sock_release(sk->sk_socket);
|
||||
sk->sk_net = get_net(&init_net);
|
||||
sock_put(sk);
|
||||
}
|
||||
|
||||
struct sock *
|
||||
netlink_kernel_create(struct net *net, int unit, unsigned int groups,
|
||||
void (*input)(struct sk_buff *skb),
|
||||
@ -1424,7 +1408,7 @@ netlink_kernel_create(struct net *net, int unit, unsigned int groups,
|
||||
|
||||
out_sock_release:
|
||||
kfree(listeners);
|
||||
__netlink_release(sk);
|
||||
netlink_kernel_release(sk);
|
||||
return NULL;
|
||||
|
||||
out_sock_release_nosk:
|
||||
@ -1437,10 +1421,20 @@ EXPORT_SYMBOL(netlink_kernel_create);
|
||||
void
|
||||
netlink_kernel_release(struct sock *sk)
|
||||
{
|
||||
/*
|
||||
* Last sock_put should drop referrence to sk->sk_net. It has already
|
||||
* been dropped in netlink_kernel_create. Taking referrence to stopping
|
||||
* namespace is not an option.
|
||||
* Take referrence to a socket to remove it from netlink lookup table
|
||||
* _alive_ and after that destroy it in the context of init_net.
|
||||
*/
|
||||
if (sk == NULL || sk->sk_socket == NULL)
|
||||
return;
|
||||
|
||||
__netlink_release(sk);
|
||||
sock_hold(sk);
|
||||
sock_release(sk->sk_socket);
|
||||
sk->sk_net = get_net(&init_net);
|
||||
sock_put(sk);
|
||||
}
|
||||
EXPORT_SYMBOL(netlink_kernel_release);
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user