提交 0c0b5ebd 编写于 作者: K Kuniyuki Iwashima 提交者: Jialin Zhang

inet6: Remove inet6_destroy_sock() in sk->sk_prot->destroy().

mainline inclusion
from mainline-v6.2-rc1
commit b5fc2923
category: bugfix
bugzilla: https://gitee.com/openeuler/kernel/issues/I6TPN9
CVE: NA

Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?h=v6.3-rc2&id=b5fc29233d28be7a3322848ebe73ac327559cdb9

--------------------------------

After commit d38afeec ("tcp/udp: Call inet6_destroy_sock()
in IPv6 sk->sk_destruct()."), we call inet6_destroy_sock() in
sk->sk_destruct() by setting inet6_sock_destruct() to it to make
sure we do not leak inet6-specific resources.

Now we can remove unnecessary inet6_destroy_sock() calls in
sk->sk_prot->destroy().

DCCP and SCTP have their own sk->sk_destruct() function, so we
change them separately in the following patches.
Signed-off-by: NKuniyuki Iwashima <kuniyu@amazon.com>
Reviewed-by: NMatthieu Baerts <matthieu.baerts@tessares.net>
Signed-off-by: NDavid S. Miller <davem@davemloft.net>
Conflicts:
	net/ipv6/ping.c
Signed-off-by: NZiyang Xuan <william.xuanziyang@huawei.com>
Reviewed-by: NLiu Jian <liujian56@huawei.com>
Signed-off-by: NJialin Zhang <zhangjialin11@huawei.com>
上级 da9581d8
...@@ -22,11 +22,6 @@ ...@@ -22,11 +22,6 @@
#include <linux/proc_fs.h> #include <linux/proc_fs.h>
#include <net/ping.h> #include <net/ping.h>
static void ping_v6_destroy(struct sock *sk)
{
inet6_destroy_sock(sk);
}
/* Compatibility glue so we can support IPv6 when it's compiled as a module */ /* Compatibility glue so we can support IPv6 when it's compiled as a module */
static int dummy_ipv6_recv_error(struct sock *sk, struct msghdr *msg, int len, static int dummy_ipv6_recv_error(struct sock *sk, struct msghdr *msg, int len,
int *addr_len) int *addr_len)
...@@ -171,7 +166,6 @@ struct proto pingv6_prot = { ...@@ -171,7 +166,6 @@ struct proto pingv6_prot = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
.init = ping_init_sock, .init = ping_init_sock,
.close = ping_close, .close = ping_close,
.destroy = ping_v6_destroy,
.connect = ip6_datagram_connect_v6_only, .connect = ip6_datagram_connect_v6_only,
.disconnect = __udp_disconnect, .disconnect = __udp_disconnect,
.setsockopt = ipv6_setsockopt, .setsockopt = ipv6_setsockopt,
......
...@@ -1177,8 +1177,6 @@ static void raw6_destroy(struct sock *sk) ...@@ -1177,8 +1177,6 @@ static void raw6_destroy(struct sock *sk)
lock_sock(sk); lock_sock(sk);
ip6_flush_pending_frames(sk); ip6_flush_pending_frames(sk);
release_sock(sk); release_sock(sk);
inet6_destroy_sock(sk);
} }
static int rawv6_init_sk(struct sock *sk) static int rawv6_init_sk(struct sock *sk)
......
...@@ -1939,12 +1939,6 @@ static int tcp_v6_init_sock(struct sock *sk) ...@@ -1939,12 +1939,6 @@ static int tcp_v6_init_sock(struct sock *sk)
return 0; return 0;
} }
static void tcp_v6_destroy_sock(struct sock *sk)
{
tcp_v4_destroy_sock(sk);
inet6_destroy_sock(sk);
}
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
/* Proc filesystem TCPv6 sock list dumping. */ /* Proc filesystem TCPv6 sock list dumping. */
static void get_openreq6(struct seq_file *seq, static void get_openreq6(struct seq_file *seq,
...@@ -2137,7 +2131,7 @@ struct proto tcpv6_prot = { ...@@ -2137,7 +2131,7 @@ struct proto tcpv6_prot = {
.accept = inet_csk_accept, .accept = inet_csk_accept,
.ioctl = tcp_ioctl, .ioctl = tcp_ioctl,
.init = tcp_v6_init_sock, .init = tcp_v6_init_sock,
.destroy = tcp_v6_destroy_sock, .destroy = tcp_v4_destroy_sock,
.shutdown = tcp_shutdown, .shutdown = tcp_shutdown,
.setsockopt = tcp_setsockopt, .setsockopt = tcp_setsockopt,
.getsockopt = tcp_getsockopt, .getsockopt = tcp_getsockopt,
......
...@@ -1627,8 +1627,6 @@ void udpv6_destroy_sock(struct sock *sk) ...@@ -1627,8 +1627,6 @@ void udpv6_destroy_sock(struct sock *sk)
udp_encap_disable(); udp_encap_disable();
} }
} }
inet6_destroy_sock(sk);
} }
/* /*
......
...@@ -255,8 +255,6 @@ static void l2tp_ip6_destroy_sock(struct sock *sk) ...@@ -255,8 +255,6 @@ static void l2tp_ip6_destroy_sock(struct sock *sk)
if (tunnel) if (tunnel)
l2tp_tunnel_delete(tunnel); l2tp_tunnel_delete(tunnel);
inet6_destroy_sock(sk);
} }
static int l2tp_ip6_bind(struct sock *sk, struct sockaddr *uaddr, int addr_len) static int l2tp_ip6_bind(struct sock *sk, struct sockaddr *uaddr, int addr_len)
......
...@@ -2863,12 +2863,6 @@ static const struct proto_ops mptcp_v6_stream_ops = { ...@@ -2863,12 +2863,6 @@ static const struct proto_ops mptcp_v6_stream_ops = {
static struct proto mptcp_v6_prot; static struct proto mptcp_v6_prot;
static void mptcp_v6_destroy(struct sock *sk)
{
mptcp_destroy(sk);
inet6_destroy_sock(sk);
}
static struct inet_protosw mptcp_v6_protosw = { static struct inet_protosw mptcp_v6_protosw = {
.type = SOCK_STREAM, .type = SOCK_STREAM,
.protocol = IPPROTO_MPTCP, .protocol = IPPROTO_MPTCP,
...@@ -2884,7 +2878,6 @@ int __init mptcp_proto_v6_init(void) ...@@ -2884,7 +2878,6 @@ int __init mptcp_proto_v6_init(void)
mptcp_v6_prot = mptcp_prot; mptcp_v6_prot = mptcp_prot;
strcpy(mptcp_v6_prot.name, "MPTCPv6"); strcpy(mptcp_v6_prot.name, "MPTCPv6");
mptcp_v6_prot.slab = NULL; mptcp_v6_prot.slab = NULL;
mptcp_v6_prot.destroy = mptcp_v6_destroy;
mptcp_v6_prot.obj_size = sizeof(struct mptcp6_sock); mptcp_v6_prot.obj_size = sizeof(struct mptcp6_sock);
err = proto_register(&mptcp_v6_prot, 1); err = proto_register(&mptcp_v6_prot, 1);
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册