bianbu-repo-mirror/bianbu-linux-6.6
1
0
Fork 0
mirror of https://gitee.com/bianbu-linux/linux-6.6 synced 2025-04-24 14:07:52 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

af_unix: Remove UNIX_ABSTRACT() macro and test sun_path[0] instead.

Browse source 
In BSD and abstract address cases, we store sockets in the hash table with
keys between 0 and UNIX_HASH_SIZE - 1.  However, the hash saved in a socket
varies depending on its address type; sockets with BSD addresses always
have UNIX_HASH_SIZE in their unix_sk(sk)->addr->hash.

This is just for the UNIX_ABSTRACT() macro used to check the address type.
The difference of the saved hashes comes from the first byte of the address
in the first place.  So, we can test it directly.

Then we can keep a real hash in each socket and replace unix_table_lock
with per-hash locks in the later patch.

Signed-off-by: Kuniyuki Iwashima <kuniyu@amazon.co.jp>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
Kuniyuki Iwashima 2021-11-24 11:14:27 +09:00 • committed by Jakub Kicinski
parent 12f21c49ad
commit 5ce7ab4961
4 changed files with 4 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

6
net/unix/af_unix.c
View file

@ -134,8 +134,6 @@ static struct hlist_head *unix_sockets_unbound(void *addr)
return &unix_socket_table[UNIX_HASH_SIZE + hash];
}
#define UNIX_ABSTRACT(sk) (unix_sk(sk)->addr->hash < UNIX_HASH_SIZE)
#ifdef CONFIG_SECURITY_NETWORK
static void unix_get_secdata(struct scm_cookie *scm, struct sk_buff *skb)
{
@ -3292,9 +3290,9 @@ static int unix_seq_show(struct seq_file *seq, void *v)
i = 0;
len = u->addr->len -
offsetof(struct sockaddr_un, sun_path);
if (!UNIX_ABSTRACT(s))
if (u->addr->name->sun_path[0]) {
len--;
else {
} else {
seq_putc(seq, '@');
i++;
}