Unverified Commit 13900cb8 authored by openeuler-ci-bot's avatar openeuler-ci-bot Committed by Gitee
Browse files

!14046 smb: client: Fix use-after-free of network namespace.

parents aa8466e7 fe889cc8
Loading
Loading
Loading
Loading
+12 −3
Original line number Diff line number Diff line
@@ -966,6 +966,7 @@ static void clean_demultiplex_info(struct TCP_Server_Info *server)
		 */
	}

	put_net(cifs_net_ns(server));
	kfree(server->hostname);
	kfree(server);

@@ -2530,8 +2531,6 @@ cifs_put_tcp_session(struct TCP_Server_Info *server, int from_reconnect)
		return;
	}

	put_net(cifs_net_ns(server));

	list_del_init(&server->tcp_ses_list);
	spin_unlock(&cifs_tcp_ses_lock);

@@ -3743,7 +3742,10 @@ generic_ip_connect(struct TCP_Server_Info *server)
	}

	if (socket == NULL) {
		rc = __sock_create(cifs_net_ns(server), sfamily, SOCK_STREAM,
		struct net *net = cifs_net_ns(server);
		struct sock *sk;

		rc = __sock_create(net, sfamily, SOCK_STREAM,
				   IPPROTO_TCP, &socket, 1);
		if (rc < 0) {
			cifs_server_dbg(VFS, "Error %d creating socket\n", rc);
@@ -3751,6 +3753,13 @@ generic_ip_connect(struct TCP_Server_Info *server)
			return rc;
		}

		sk = socket->sk;
		sk->sk_net_refcnt = 1;
		get_net(net);
#ifdef CONFIG_PROC_FS
		this_cpu_add(*net->core.sock_inuse, 1);
#endif

		/* BB other socket options to set KEEPALIVE, NODELAY? */
		cifs_dbg(FYI, "Socket created\n");
		server->ssocket = socket;