[PATCH 2/8] unix_bind(): allocate addr earlier

[Al Viro]

makes it easier to massage; we do pay for that by extra work
(kmalloc+memcpy+kfree) in some error cases, but those are not
on the hot paths anyway.

Signed-off-by: Al Viro <v...@zeniv.linux.org.uk>
---
 net/unix/af_unix.c | 28 +++++++++++++++-------------
 1 file changed, 15 insertions(+), 13 deletions(-)

diff --git a/net/unix/af_unix.c b/net/unix/af_unix.c
index 45a40cf7b6af..54f1bfe14191 100644
--- a/net/unix/af_unix.c
+++ b/net/unix/af_unix.c
@@ -1038,6 +1038,15 @@ static int unix_bind(struct socket *sock, struct 
sockaddr *uaddr, int addr_len)
        if (err < 0)
                goto out;
        addr_len = err;
+       err = -ENOMEM;
+       addr = kmalloc(sizeof(*addr)+addr_len, GFP_KERNEL);
+       if (!addr)
+               goto out;
+
+       memcpy(addr->name, sunaddr, addr_len);
+       addr->len = addr_len;
+       addr->hash = hash ^ sk->sk_type;
+       refcount_set(&addr->refcnt, 1);
 
        if (sun_path[0]) {
                umode_t mode = S_IFSOCK |
@@ -1046,7 +1055,7 @@ static int unix_bind(struct socket *sock, struct sockaddr 
*uaddr, int addr_len)
                if (err) {
                        if (err == -EEXIST)
                                err = -EADDRINUSE;
-                       goto out;
+                       goto out_addr;
                }
        }
 
@@ -1058,16 +1067,6 @@ static int unix_bind(struct socket *sock, struct 
sockaddr *uaddr, int addr_len)
        if (u->addr)
                goto out_up;
 
-       err = -ENOMEM;
-       addr = kmalloc(sizeof(*addr)+addr_len, GFP_KERNEL);
-       if (!addr)
-               goto out_up;
-
-       memcpy(addr->name, sunaddr, addr_len);
-       addr->len = addr_len;
-       addr->hash = hash ^ sk->sk_type;
-       refcount_set(&addr->refcnt, 1);
-
        if (sun_path[0]) {
                addr->hash = UNIX_HASH_SIZE;
                hash = d_backing_inode(path.dentry)->i_ino & (UNIX_HASH_SIZE - 
1);
@@ -1079,20 +1078,23 @@ static int unix_bind(struct socket *sock, struct 
sockaddr *uaddr, int addr_len)
                if (__unix_find_socket_byname(net, sunaddr, addr_len,
                                              sk->sk_type, hash)) {
                        spin_unlock(&unix_table_lock);
-                       unix_release_addr(addr);
                        goto out_up;
                }
                hash = addr->hash;
        }
 
-       err = 0;
        __unix_set_addr(sk, addr, hash);
        spin_unlock(&unix_table_lock);
+       addr = NULL;
+       err = 0;
 out_up:
        mutex_unlock(&u->bindlock);
 out_put:
        if (err)
                path_put(&path);
+out_addr:
+       if (addr)
+               unix_release_addr(addr);
 out:
        return err;
 }
-- 
2.11.0