Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d623fd52 authored by Jason Xing's avatar Jason Xing Committed by Sasha Levin
Browse files

netrom: Fix data-races around sysctl_net_busy_read 



[ Upstream commit d380ce70058a4ccddc3e5f5c2063165dc07672c6 ]

We need to protect the reader reading the sysctl value because the
value can be changed concurrently.

Fixes: 1da177e4 ("Linux-2.6.12-rc2")
Signed-off-by: default avatarJason Xing <kernelxing@tencent.com>
Signed-off-by: default avatarPaolo Abeni <pabeni@redhat.com>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent 97a4d8b9

net/netrom/af_netrom.c

+1 −1
Original line number Diff line number Diff line
@@ -957,7 +957,7 @@ int nr_rx_frame(struct sk_buff *skb, struct net_device *dev) 
		 * G8PZT's Xrouter which is sending packets with command type 7

		 * as an extension of the protocol.

		 */

		if (sysctl_netrom_reset_circuit &&

		if (READ_ONCE(sysctl_netrom_reset_circuit) &&

		    (frametype != NR_RESET || flags != 0))

			nr_transmit_reset(skb, 1);

net/netrom/nr_in.c

+3 −3
Original line number Diff line number Diff line
@@ -100,7 +100,7 @@ static int nr_state1_machine(struct sock *sk, struct sk_buff *skb, 
		break;



	case NR_RESET:

		if (sysctl_netrom_reset_circuit)

		if (READ_ONCE(sysctl_netrom_reset_circuit))

			nr_disconnect(sk, ECONNRESET);

		break;
@@ -131,7 +131,7 @@ static int nr_state2_machine(struct sock *sk, struct sk_buff *skb, 
		break;



	case NR_RESET:

		if (sysctl_netrom_reset_circuit)

		if (READ_ONCE(sysctl_netrom_reset_circuit))

			nr_disconnect(sk, ECONNRESET);

		break;
@@ -266,7 +266,7 @@ static int nr_state3_machine(struct sock *sk, struct sk_buff *skb, int frametype 
		break;



	case NR_RESET:

		if (sysctl_netrom_reset_circuit)

		if (READ_ONCE(sysctl_netrom_reset_circuit))

			nr_disconnect(sk, ECONNRESET);

		break;