net: ignore sock_from_file errors in __scm_install_fd
authorChristoph Hellwig <hch@lst.de>
Wed, 13 May 2020 11:07:59 +0000 (13:07 +0200)
committerDavid S. Miller <davem@davemloft.net>
Wed, 13 May 2020 19:30:54 +0000 (12:30 -0700)
The code had historically been ignoring these errors, and my recent
refactoring changed that, which broke ssh in some setups.

Fixes: 2618d530dd8b ("net/scm: cleanup scm_detach_fds")
Reported-by: Ido Schimmel <idosch@idosch.org>
Signed-off-by: Christoph Hellwig <hch@lst.de>
Tested-by: Ido Schimmel <idosch@mellanox.com>
Tested-by: Ioana Ciornei <ioana.ciornei@nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/scm.c

index a75cd63..875df1c 100644 (file)
@@ -307,7 +307,7 @@ static int __scm_install_fd(struct file *file, int __user *ufd, int o_flags)
                sock_update_classid(&sock->sk->sk_cgrp_data);
        }
        fd_install(new_fd, get_file(file));
-       return error;
+       return 0;
 }
 
 static int scm_max_fds(struct msghdr *msg)