Commit 6059ddb2 authored Jun 26, 2014 by Vinay Krishna Eranna Committed by Gerrit - the friendly Code Review server Jul 04, 2014

netlink: Queue the kernel socket after setting the flag.



Queueing the socket after setting the NETLINK_KERNEL_SOCKET on the
kernel socket.

This change is required in-order to avoid the BUG check which is
caused due to race condition between setting this flag and a
message from the app space for this kernel netlink sock.

Change-Id: I19a8edf2fe009a3020b194684a6172654f8f257a
CRs-Fixed: 681815
Signed-off-by: Vinay Krishna Eranna <veran@codeaurora.org>

parent dee710f6

net/netlink/af_netlink.c

+3 −3

Original line number	Diff line number	Diff line
		@@ -2268,12 +2268,12 @@ __netlink_kernel_create(struct net net, int unit, struct module module,
		if (cfg && cfg->input)
		nlk_sk(sk)->netlink_rcv = cfg->input;

		if (netlink_insert(sk, net, 0))
		goto out_sock_release;

		nlk = nlk_sk(sk);
		nlk->flags \|= NETLINK_KERNEL_SOCKET;

		if (netlink_insert(sk, net, 0))
		goto out_sock_release;

		netlink_table_grab();
		if (!nl_table[unit].registered) {
		nl_table[unit].groups = groups;