Commit 8e1611e2 authored by Al Viro's avatar Al Viro Committed by David S. Miller

make sock_alloc_file() do sock_release() on failures

This changes calling conventions (and simplifies the hell out
the callers).  New rules: once struct socket had been passed
to sock_alloc_file(), it's been consumed either by struct file
or by sock_release() done by sock_alloc_file().  Either way
the caller should not do sock_release() after that point.
Reviewed-by: default avatarEric Dumazet <edumazet@google.com>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 016a266b
...@@ -71,16 +71,12 @@ lnet_sock_ioctl(int cmd, unsigned long arg) ...@@ -71,16 +71,12 @@ lnet_sock_ioctl(int cmd, unsigned long arg)
} }
sock_filp = sock_alloc_file(sock, 0, NULL); sock_filp = sock_alloc_file(sock, 0, NULL);
if (IS_ERR(sock_filp)) { if (IS_ERR(sock_filp))
sock_release(sock); return PTR_ERR(sock_filp);
rc = PTR_ERR(sock_filp);
goto out;
}
rc = kernel_sock_unlocked_ioctl(sock_filp, cmd, arg); rc = kernel_sock_unlocked_ioctl(sock_filp, cmd, arg);
fput(sock_filp); fput(sock_filp);
out:
return rc; return rc;
} }
......
...@@ -839,7 +839,6 @@ static int p9_socket_open(struct p9_client *client, struct socket *csocket) ...@@ -839,7 +839,6 @@ static int p9_socket_open(struct p9_client *client, struct socket *csocket)
if (IS_ERR(file)) { if (IS_ERR(file)) {
pr_err("%s (%d): failed to map fd\n", pr_err("%s (%d): failed to map fd\n",
__func__, task_pid_nr(current)); __func__, task_pid_nr(current));
sock_release(csocket);
kfree(p); kfree(p);
return PTR_ERR(file); return PTR_ERR(file);
} }
......
...@@ -1629,7 +1629,6 @@ static struct file *kcm_clone(struct socket *osock) ...@@ -1629,7 +1629,6 @@ static struct file *kcm_clone(struct socket *osock)
{ {
struct socket *newsock; struct socket *newsock;
struct sock *newsk; struct sock *newsk;
struct file *file;
newsock = sock_alloc(); newsock = sock_alloc();
if (!newsock) if (!newsock)
...@@ -1649,11 +1648,7 @@ static struct file *kcm_clone(struct socket *osock) ...@@ -1649,11 +1648,7 @@ static struct file *kcm_clone(struct socket *osock)
sock_init_data(newsock, newsk); sock_init_data(newsock, newsk);
init_kcm_sock(kcm_sk(newsk), kcm_sk(osock->sk)->mux); init_kcm_sock(kcm_sk(newsk), kcm_sk(osock->sk)->mux);
file = sock_alloc_file(newsock, 0, osock->sk->sk_prot_creator->name); return sock_alloc_file(newsock, 0, osock->sk->sk_prot_creator->name);
if (IS_ERR(file))
sock_release(newsock);
return file;
} }
static int kcm_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg) static int kcm_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg)
......
...@@ -5080,7 +5080,6 @@ static int sctp_getsockopt_peeloff_common(struct sock *sk, sctp_peeloff_arg_t *p ...@@ -5080,7 +5080,6 @@ static int sctp_getsockopt_peeloff_common(struct sock *sk, sctp_peeloff_arg_t *p
*newfile = sock_alloc_file(newsock, 0, NULL); *newfile = sock_alloc_file(newsock, 0, NULL);
if (IS_ERR(*newfile)) { if (IS_ERR(*newfile)) {
put_unused_fd(retval); put_unused_fd(retval);
sock_release(newsock);
retval = PTR_ERR(*newfile); retval = PTR_ERR(*newfile);
*newfile = NULL; *newfile = NULL;
return retval; return retval;
......
...@@ -406,8 +406,10 @@ struct file *sock_alloc_file(struct socket *sock, int flags, const char *dname) ...@@ -406,8 +406,10 @@ struct file *sock_alloc_file(struct socket *sock, int flags, const char *dname)
name.len = strlen(name.name); name.len = strlen(name.name);
} }
path.dentry = d_alloc_pseudo(sock_mnt->mnt_sb, &name); path.dentry = d_alloc_pseudo(sock_mnt->mnt_sb, &name);
if (unlikely(!path.dentry)) if (unlikely(!path.dentry)) {
sock_release(sock);
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
}
path.mnt = mntget(sock_mnt); path.mnt = mntget(sock_mnt);
d_instantiate(path.dentry, SOCK_INODE(sock)); d_instantiate(path.dentry, SOCK_INODE(sock));
...@@ -415,9 +417,11 @@ struct file *sock_alloc_file(struct socket *sock, int flags, const char *dname) ...@@ -415,9 +417,11 @@ struct file *sock_alloc_file(struct socket *sock, int flags, const char *dname)
file = alloc_file(&path, FMODE_READ | FMODE_WRITE, file = alloc_file(&path, FMODE_READ | FMODE_WRITE,
&socket_file_ops); &socket_file_ops);
if (IS_ERR(file)) { if (IS_ERR(file)) {
/* drop dentry, keep inode */ /* drop dentry, keep inode for a bit */
ihold(d_inode(path.dentry)); ihold(d_inode(path.dentry));
path_put(&path); path_put(&path);
/* ... and now kill it properly */
sock_release(sock);
return file; return file;
} }
...@@ -1330,19 +1334,9 @@ SYSCALL_DEFINE3(socket, int, family, int, type, int, protocol) ...@@ -1330,19 +1334,9 @@ SYSCALL_DEFINE3(socket, int, family, int, type, int, protocol)
retval = sock_create(family, type, protocol, &sock); retval = sock_create(family, type, protocol, &sock);
if (retval < 0) if (retval < 0)
goto out; return retval;
retval = sock_map_fd(sock, flags & (O_CLOEXEC | O_NONBLOCK));
if (retval < 0)
goto out_release;
out:
/* It may be already another descriptor 8) Not kernel problem. */
return retval;
out_release: return sock_map_fd(sock, flags & (O_CLOEXEC | O_NONBLOCK));
sock_release(sock);
return retval;
} }
/* /*
...@@ -1412,7 +1406,6 @@ SYSCALL_DEFINE4(socketpair, int, family, int, type, int, protocol, ...@@ -1412,7 +1406,6 @@ SYSCALL_DEFINE4(socketpair, int, family, int, type, int, protocol,
newfile1 = sock_alloc_file(sock1, flags, NULL); newfile1 = sock_alloc_file(sock1, flags, NULL);
if (IS_ERR(newfile1)) { if (IS_ERR(newfile1)) {
err = PTR_ERR(newfile1); err = PTR_ERR(newfile1);
sock_release(sock1);
sock_release(sock2); sock_release(sock2);
goto out; goto out;
} }
...@@ -1420,7 +1413,6 @@ SYSCALL_DEFINE4(socketpair, int, family, int, type, int, protocol, ...@@ -1420,7 +1413,6 @@ SYSCALL_DEFINE4(socketpair, int, family, int, type, int, protocol,
newfile2 = sock_alloc_file(sock2, flags, NULL); newfile2 = sock_alloc_file(sock2, flags, NULL);
if (IS_ERR(newfile2)) { if (IS_ERR(newfile2)) {
err = PTR_ERR(newfile2); err = PTR_ERR(newfile2);
sock_release(sock2);
fput(newfile1); fput(newfile1);
goto out; goto out;
} }
...@@ -1549,7 +1541,6 @@ SYSCALL_DEFINE4(accept4, int, fd, struct sockaddr __user *, upeer_sockaddr, ...@@ -1549,7 +1541,6 @@ SYSCALL_DEFINE4(accept4, int, fd, struct sockaddr __user *, upeer_sockaddr,
if (IS_ERR(newfile)) { if (IS_ERR(newfile)) {
err = PTR_ERR(newfile); err = PTR_ERR(newfile);
put_unused_fd(newfd); put_unused_fd(newfd);
sock_release(newsock);
goto out_put; goto out_put;
} }
......
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