Skip to content

Commit

Permalink
inet6: Remove inet6_destroy_sock() in sk->sk_prot->destroy().
Browse files Browse the repository at this point in the history
commit b5fc292 upstream.

After commit d38afee ("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: Kuniyuki Iwashima <kuniyu@amazon.com>
Reviewed-by: Matthieu Baerts <matthieu.baerts@tessares.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Ziyang Xuan <william.xuanziyang@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
  • Loading branch information
Kuniyuki Iwashima authored and Greg Kroah-Hartman committed Apr 26, 2023
1 parent 1d8a87d commit e1820a9
Show file tree
Hide file tree
Showing 5 changed files with 1 addition and 19 deletions.
6 changes: 0 additions & 6 deletions net/ipv6/ping.c
Original file line number Diff line number Diff line change
Expand Up @@ -27,11 +27,6 @@
#include <linux/proc_fs.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 */
static int dummy_ipv6_recv_error(struct sock *sk, struct msghdr *msg, int len,
int *addr_len)
Expand Down Expand Up @@ -175,7 +170,6 @@ struct proto pingv6_prot = {
.owner = THIS_MODULE,
.init = ping_init_sock,
.close = ping_close,
.destroy = ping_v6_destroy,
.connect = ip6_datagram_connect_v6_only,
.disconnect = __udp_disconnect,
.setsockopt = ipv6_setsockopt,
Expand Down
2 changes: 0 additions & 2 deletions net/ipv6/raw.c
Original file line number Diff line number Diff line change
Expand Up @@ -1259,8 +1259,6 @@ static void raw6_destroy(struct sock *sk)
lock_sock(sk);
ip6_flush_pending_frames(sk);
release_sock(sk);

inet6_destroy_sock(sk);
}

static int rawv6_init_sk(struct sock *sk)
Expand Down
8 changes: 1 addition & 7 deletions net/ipv6/tcp_ipv6.c
Original file line number Diff line number Diff line change
Expand Up @@ -1792,12 +1792,6 @@ static int tcp_v6_init_sock(struct sock *sk)
return 0;
}

static void tcp_v6_destroy_sock(struct sock *sk)
{
tcp_v4_destroy_sock(sk);
inet6_destroy_sock(sk);
}

#ifdef CONFIG_PROC_FS
/* Proc filesystem TCPv6 sock list dumping. */
static void get_openreq6(struct seq_file *seq,
Expand Down Expand Up @@ -1990,7 +1984,7 @@ struct proto tcpv6_prot = {
.accept = inet_csk_accept,
.ioctl = tcp_ioctl,
.init = tcp_v6_init_sock,
.destroy = tcp_v6_destroy_sock,
.destroy = tcp_v4_destroy_sock,
.shutdown = tcp_shutdown,
.setsockopt = tcp_setsockopt,
.getsockopt = tcp_getsockopt,
Expand Down
2 changes: 0 additions & 2 deletions net/ipv6/udp.c
Original file line number Diff line number Diff line change
Expand Up @@ -1503,8 +1503,6 @@ void udpv6_destroy_sock(struct sock *sk)
if (encap_destroy)
encap_destroy(sk);
}

inet6_destroy_sock(sk);
}

/*
Expand Down
2 changes: 0 additions & 2 deletions net/l2tp/l2tp_ip6.c
Original file line number Diff line number Diff line change
Expand Up @@ -272,8 +272,6 @@ static void l2tp_ip6_destroy_sock(struct sock *sk)

if (tunnel)
l2tp_tunnel_delete(tunnel);

inet6_destroy_sock(sk);
}

static int l2tp_ip6_bind(struct sock *sk, struct sockaddr *uaddr, int addr_len)
Expand Down

0 comments on commit e1820a9

Please sign in to comment.