Commit b8a58aa6 authored by Kuniyuki Iwashima's avatar Kuniyuki Iwashima Committed by Jakub Kicinski

af_unix: Cut unix_validate_addr() out of unix_mkname().

unix_mkname() tests socket address length and family and does some
processing based on the address type.  It is called in the early stage,
and therefore some instructions are redundant and can end up in vain.

The address length/family tests are done twice in unix_bind().  Also, the
address type is rechecked later in unix_bind() and unix_find_other(), where
we can do the same processing.  Moreover, in the BSD address case, the hash
is set to 0 but never used and confusing.

This patch moves the address tests out of unix_mkname(), and the following
patches move the other part into appropriate places and remove
unix_mkname() finally.
Signed-off-by: default avatarKuniyuki Iwashima <kuniyu@amazon.co.jp>
Signed-off-by: default avatarJakub Kicinski <kuba@kernel.org>
parent aed26f55
...@@ -227,15 +227,22 @@ static inline void unix_release_addr(struct unix_address *addr) ...@@ -227,15 +227,22 @@ static inline void unix_release_addr(struct unix_address *addr)
* - if started by zero, it is abstract name. * - if started by zero, it is abstract name.
*/ */
static int unix_validate_addr(struct sockaddr_un *sunaddr, int addr_len)
{
if (addr_len <= offsetof(struct sockaddr_un, sun_path) ||
addr_len > sizeof(*sunaddr))
return -EINVAL;
if (sunaddr->sun_family != AF_UNIX)
return -EINVAL;
return 0;
}
static int unix_mkname(struct sockaddr_un *sunaddr, int len, unsigned int *hashp) static int unix_mkname(struct sockaddr_un *sunaddr, int len, unsigned int *hashp)
{ {
*hashp = 0; *hashp = 0;
if (len <= offsetof(struct sockaddr_un, sun_path) ||
len > sizeof(*sunaddr))
return -EINVAL;
if (!sunaddr || sunaddr->sun_family != AF_UNIX)
return -EINVAL;
if (sunaddr->sun_path[0]) { if (sunaddr->sun_path[0]) {
/* /*
* This may look like an off by one error but it is a bit more * This may look like an off by one error but it is a bit more
...@@ -1177,13 +1184,14 @@ static int unix_bind(struct socket *sock, struct sockaddr *uaddr, int addr_len) ...@@ -1177,13 +1184,14 @@ static int unix_bind(struct socket *sock, struct sockaddr *uaddr, int addr_len)
unsigned int hash; unsigned int hash;
struct unix_address *addr; struct unix_address *addr;
if (addr_len < offsetofend(struct sockaddr_un, sun_family) || if (addr_len == offsetof(struct sockaddr_un, sun_path) &&
sunaddr->sun_family != AF_UNIX) sunaddr->sun_family == AF_UNIX)
return -EINVAL;
if (addr_len == offsetof(struct sockaddr_un, sun_path))
return unix_autobind(sk); return unix_autobind(sk);
err = unix_validate_addr(sunaddr, addr_len);
if (err)
return err;
err = unix_mkname(sunaddr, addr_len, &hash); err = unix_mkname(sunaddr, addr_len, &hash);
if (err < 0) if (err < 0)
return err; return err;
...@@ -1246,6 +1254,10 @@ static int unix_dgram_connect(struct socket *sock, struct sockaddr *addr, ...@@ -1246,6 +1254,10 @@ static int unix_dgram_connect(struct socket *sock, struct sockaddr *addr,
goto out; goto out;
if (addr->sa_family != AF_UNSPEC) { if (addr->sa_family != AF_UNSPEC) {
err = unix_validate_addr(sunaddr, alen);
if (err)
goto out;
err = unix_mkname(sunaddr, alen, &hash); err = unix_mkname(sunaddr, alen, &hash);
if (err < 0) if (err < 0)
goto out; goto out;
...@@ -1358,6 +1370,10 @@ static int unix_stream_connect(struct socket *sock, struct sockaddr *uaddr, ...@@ -1358,6 +1370,10 @@ static int unix_stream_connect(struct socket *sock, struct sockaddr *uaddr,
int err; int err;
long timeo; long timeo;
err = unix_validate_addr(sunaddr, addr_len);
if (err)
goto out;
err = unix_mkname(sunaddr, addr_len, &hash); err = unix_mkname(sunaddr, addr_len, &hash);
if (err < 0) if (err < 0)
goto out; goto out;
...@@ -1810,6 +1826,10 @@ static int unix_dgram_sendmsg(struct socket *sock, struct msghdr *msg, ...@@ -1810,6 +1826,10 @@ static int unix_dgram_sendmsg(struct socket *sock, struct msghdr *msg,
goto out; goto out;
if (msg->msg_namelen) { if (msg->msg_namelen) {
err = unix_validate_addr(sunaddr, msg->msg_namelen);
if (err)
goto out;
err = unix_mkname(sunaddr, msg->msg_namelen, &hash); err = unix_mkname(sunaddr, msg->msg_namelen, &hash);
if (err < 0) if (err < 0)
goto out; goto out;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment