]> git.proxmox.com Git - mirror_ubuntu-focal-kernel.git/blobdiff - net/socket.c
tuntap: add sanity checks about msg_controllen in sendmsg
[mirror_ubuntu-focal-kernel.git] / net / socket.c
index 6a9ab7a8b1d2c5059752a9147c2f0fd1aea4f44d..94358566c9d10b564e94455ef7320ba9752e63ae 100644 (file)
@@ -485,7 +485,7 @@ static struct socket *sockfd_lookup_light(int fd, int *err, int *fput_needed)
        if (f.file) {
                sock = sock_from_file(f.file, err);
                if (likely(sock)) {
-                       *fput_needed = f.flags;
+                       *fput_needed = f.flags & FDPUT_FPUT;
                        return sock;
                }
                fdput(f);
@@ -955,7 +955,7 @@ static ssize_t sock_read_iter(struct kiocb *iocb, struct iov_iter *to)
                             .msg_iocb = iocb};
        ssize_t res;
 
-       if (file->f_flags & O_NONBLOCK)
+       if (file->f_flags & O_NONBLOCK || (iocb->ki_flags & IOCB_NOWAIT))
                msg.msg_flags = MSG_DONTWAIT;
 
        if (iocb->ki_pos != 0)
@@ -980,7 +980,7 @@ static ssize_t sock_write_iter(struct kiocb *iocb, struct iov_iter *from)
        if (iocb->ki_pos != 0)
                return -ESPIPE;
 
-       if (file->f_flags & O_NONBLOCK)
+       if (file->f_flags & O_NONBLOCK || (iocb->ki_flags & IOCB_NOWAIT))
                msg.msg_flags = MSG_DONTWAIT;
 
        if (sock->type == SOCK_SEQPACKET)
@@ -1053,7 +1053,7 @@ static long sock_do_ioctl(struct net *net, struct socket *sock,
                rtnl_unlock();
                if (!err && copy_to_user(argp, &ifc, sizeof(struct ifconf)))
                        err = -EFAULT;
-       } else {
+       } else if (is_socket_ioctl_cmd(cmd)) {
                struct ifreq ifr;
                bool need_copyout;
                if (copy_from_user(&ifr, argp, sizeof(struct ifreq)))
@@ -1062,6 +1062,8 @@ static long sock_do_ioctl(struct net *net, struct socket *sock,
                if (!err && need_copyout)
                        if (copy_to_user(argp, &ifr, sizeof(struct ifreq)))
                                return -EFAULT;
+       } else {
+               err = -ENOTTY;
        }
        return err;
 }
@@ -1071,19 +1073,6 @@ static long sock_do_ioctl(struct net *net, struct socket *sock,
  *     what to do with it - that's up to the protocol still.
  */
 
-/**
- *     get_net_ns - increment the refcount of the network namespace
- *     @ns: common namespace (net)
- *
- *     Returns the net's common namespace.
- */
-
-struct ns_common *get_net_ns(struct ns_common *ns)
-{
-       return &get_net(container_of(ns, struct net, ns))->ns;
-}
-EXPORT_SYMBOL_GPL(get_net_ns);
-
 static long sock_ioctl(struct file *file, unsigned cmd, unsigned long arg)
 {
        struct socket *sock;
@@ -2232,15 +2221,10 @@ static int copy_msghdr_from_user(struct msghdr *kmsg,
        return err < 0 ? err : 0;
 }
 
-static int ___sys_sendmsg(struct socket *sock, struct user_msghdr __user *msg,
-                        struct msghdr *msg_sys, unsigned int flags,
-                        struct used_address *used_address,
-                        unsigned int allowed_msghdr_flags)
+static int ____sys_sendmsg(struct socket *sock, struct msghdr *msg_sys,
+                          unsigned int flags, struct used_address *used_address,
+                          unsigned int allowed_msghdr_flags)
 {
-       struct compat_msghdr __user *msg_compat =
-           (struct compat_msghdr __user *)msg;
-       struct sockaddr_storage address;
-       struct iovec iovstack[UIO_FASTIOV], *iov = iovstack;
        unsigned char ctl[sizeof(struct cmsghdr) + 20]
                                __aligned(sizeof(__kernel_size_t));
        /* 20 is size of ipv6_pktinfo */
@@ -2248,19 +2232,10 @@ static int ___sys_sendmsg(struct socket *sock, struct user_msghdr __user *msg,
        int ctl_len;
        ssize_t err;
 
-       msg_sys->msg_name = &address;
-
-       if (MSG_CMSG_COMPAT & flags)
-               err = get_compat_msghdr(msg_sys, msg_compat, NULL, &iov);
-       else
-               err = copy_msghdr_from_user(msg_sys, msg, NULL, &iov);
-       if (err < 0)
-               return err;
-
        err = -ENOBUFS;
 
        if (msg_sys->msg_controllen > INT_MAX)
-               goto out_freeiov;
+               goto out;
        flags |= (msg_sys->msg_flags & allowed_msghdr_flags);
        ctl_len = msg_sys->msg_controllen;
        if ((MSG_CMSG_COMPAT & flags) && ctl_len) {
@@ -2268,7 +2243,7 @@ static int ___sys_sendmsg(struct socket *sock, struct user_msghdr __user *msg,
                    cmsghdr_from_user_compat_to_kern(msg_sys, sock->sk, ctl,
                                                     sizeof(ctl));
                if (err)
-                       goto out_freeiov;
+                       goto out;
                ctl_buf = msg_sys->msg_control;
                ctl_len = msg_sys->msg_controllen;
        } else if (ctl_len) {
@@ -2277,7 +2252,7 @@ static int ___sys_sendmsg(struct socket *sock, struct user_msghdr __user *msg,
                if (ctl_len > sizeof(ctl)) {
                        ctl_buf = sock_kmalloc(sock->sk, ctl_len, GFP_KERNEL);
                        if (ctl_buf == NULL)
-                               goto out_freeiov;
+                               goto out;
                }
                err = -EFAULT;
                /*
@@ -2323,7 +2298,47 @@ static int ___sys_sendmsg(struct socket *sock, struct user_msghdr __user *msg,
 out_freectl:
        if (ctl_buf != ctl)
                sock_kfree_s(sock->sk, ctl_buf, ctl_len);
-out_freeiov:
+out:
+       return err;
+}
+
+static int sendmsg_copy_msghdr(struct msghdr *msg,
+                              struct user_msghdr __user *umsg, unsigned flags,
+                              struct iovec **iov)
+{
+       int err;
+
+       if (flags & MSG_CMSG_COMPAT) {
+               struct compat_msghdr __user *msg_compat;
+
+               msg_compat = (struct compat_msghdr __user *) umsg;
+               err = get_compat_msghdr(msg, msg_compat, NULL, iov);
+       } else {
+               err = copy_msghdr_from_user(msg, umsg, NULL, iov);
+       }
+       if (err < 0)
+               return err;
+
+       return 0;
+}
+
+static int ___sys_sendmsg(struct socket *sock, struct user_msghdr __user *msg,
+                        struct msghdr *msg_sys, unsigned int flags,
+                        struct used_address *used_address,
+                        unsigned int allowed_msghdr_flags)
+{
+       struct sockaddr_storage address;
+       struct iovec iovstack[UIO_FASTIOV], *iov = iovstack;
+       ssize_t err;
+
+       msg_sys->msg_name = &address;
+
+       err = sendmsg_copy_msghdr(msg_sys, msg, flags, &iov);
+       if (err < 0)
+               return err;
+
+       err = ____sys_sendmsg(sock, msg_sys, flags, used_address,
+                               allowed_msghdr_flags);
        kfree(iov);
        return err;
 }
@@ -2331,12 +2346,27 @@ out_freeiov:
 /*
  *     BSD sendmsg interface
  */
-long __sys_sendmsg_sock(struct socket *sock, struct user_msghdr __user *msg,
+long __sys_sendmsg_sock(struct socket *sock, struct user_msghdr __user *umsg,
                        unsigned int flags)
 {
-       struct msghdr msg_sys;
+       struct iovec iovstack[UIO_FASTIOV], *iov = iovstack;
+       struct sockaddr_storage address;
+       struct msghdr msg = { .msg_name = &address };
+       ssize_t err;
+
+       err = sendmsg_copy_msghdr(&msg, umsg, flags, &iov);
+       if (err)
+               return err;
+       /* disallow ancillary data requests from this path */
+       if (msg.msg_control || msg.msg_controllen) {
+               err = -EINVAL;
+               goto out;
+       }
 
-       return ___sys_sendmsg(sock, msg, &msg_sys, flags, NULL, 0);
+       err = ____sys_sendmsg(sock, &msg, flags, NULL, 0);
+out:
+       kfree(iov);
+       return err;
 }
 
 long __sys_sendmsg(int fd, struct user_msghdr __user *msg, unsigned int flags,
@@ -2442,33 +2472,41 @@ SYSCALL_DEFINE4(sendmmsg, int, fd, struct mmsghdr __user *, mmsg,
        return __sys_sendmmsg(fd, mmsg, vlen, flags, true);
 }
 
-static int ___sys_recvmsg(struct socket *sock, struct user_msghdr __user *msg,
-                        struct msghdr *msg_sys, unsigned int flags, int nosec)
+static int recvmsg_copy_msghdr(struct msghdr *msg,
+                              struct user_msghdr __user *umsg, unsigned flags,
+                              struct sockaddr __user **uaddr,
+                              struct iovec **iov)
 {
-       struct compat_msghdr __user *msg_compat =
-           (struct compat_msghdr __user *)msg;
-       struct iovec iovstack[UIO_FASTIOV];
-       struct iovec *iov = iovstack;
-       unsigned long cmsg_ptr;
-       int len;
        ssize_t err;
 
-       /* kernel mode address */
-       struct sockaddr_storage addr;
-
-       /* user mode address pointers */
-       struct sockaddr __user *uaddr;
-       int __user *uaddr_len = COMPAT_NAMELEN(msg);
-
-       msg_sys->msg_name = &addr;
+       if (MSG_CMSG_COMPAT & flags) {
+               struct compat_msghdr __user *msg_compat;
 
-       if (MSG_CMSG_COMPAT & flags)
-               err = get_compat_msghdr(msg_sys, msg_compat, &uaddr, &iov);
-       else
-               err = copy_msghdr_from_user(msg_sys, msg, &uaddr, &iov);
+               msg_compat = (struct compat_msghdr __user *) umsg;
+               err = get_compat_msghdr(msg, msg_compat, uaddr, iov);
+       } else {
+               err = copy_msghdr_from_user(msg, umsg, uaddr, iov);
+       }
        if (err < 0)
                return err;
 
+       return 0;
+}
+
+static int ____sys_recvmsg(struct socket *sock, struct msghdr *msg_sys,
+                          struct user_msghdr __user *msg,
+                          struct sockaddr __user *uaddr,
+                          unsigned int flags, int nosec)
+{
+       struct compat_msghdr __user *msg_compat =
+                                       (struct compat_msghdr __user *) msg;
+       int __user *uaddr_len = COMPAT_NAMELEN(msg);
+       struct sockaddr_storage addr;
+       unsigned long cmsg_ptr;
+       int len;
+       ssize_t err;
+
+       msg_sys->msg_name = &addr;
        cmsg_ptr = (unsigned long)msg_sys->msg_control;
        msg_sys->msg_flags = flags & (MSG_CMSG_CLOEXEC|MSG_CMSG_COMPAT);
 
@@ -2479,7 +2517,7 @@ static int ___sys_recvmsg(struct socket *sock, struct user_msghdr __user *msg,
                flags |= MSG_DONTWAIT;
        err = (nosec ? sock_recvmsg_nosec : sock_recvmsg)(sock, msg_sys, flags);
        if (err < 0)
-               goto out_freeiov;
+               goto out;
        len = err;
 
        if (uaddr != NULL) {
@@ -2487,12 +2525,12 @@ static int ___sys_recvmsg(struct socket *sock, struct user_msghdr __user *msg,
                                        msg_sys->msg_namelen, uaddr,
                                        uaddr_len);
                if (err < 0)
-                       goto out_freeiov;
+                       goto out;
        }
        err = __put_user((msg_sys->msg_flags & ~MSG_CMSG_COMPAT),
                         COMPAT_FLAGS(msg));
        if (err)
-               goto out_freeiov;
+               goto out;
        if (MSG_CMSG_COMPAT & flags)
                err = __put_user((unsigned long)msg_sys->msg_control - cmsg_ptr,
                                 &msg_compat->msg_controllen);
@@ -2500,10 +2538,25 @@ static int ___sys_recvmsg(struct socket *sock, struct user_msghdr __user *msg,
                err = __put_user((unsigned long)msg_sys->msg_control - cmsg_ptr,
                                 &msg->msg_controllen);
        if (err)
-               goto out_freeiov;
+               goto out;
        err = len;
+out:
+       return err;
+}
+
+static int ___sys_recvmsg(struct socket *sock, struct user_msghdr __user *msg,
+                        struct msghdr *msg_sys, unsigned int flags, int nosec)
+{
+       struct iovec iovstack[UIO_FASTIOV], *iov = iovstack;
+       /* user mode address pointers */
+       struct sockaddr __user *uaddr;
+       ssize_t err;
 
-out_freeiov:
+       err = recvmsg_copy_msghdr(msg_sys, msg, flags, &uaddr, &iov);
+       if (err < 0)
+               return err;
+
+       err = ____sys_recvmsg(sock, msg_sys, msg, uaddr, flags, nosec);
        kfree(iov);
        return err;
 }
@@ -2512,12 +2565,28 @@ out_freeiov:
  *     BSD recvmsg interface
  */
 
-long __sys_recvmsg_sock(struct socket *sock, struct user_msghdr __user *msg,
+long __sys_recvmsg_sock(struct socket *sock, struct user_msghdr __user *umsg,
                        unsigned int flags)
 {
-       struct msghdr msg_sys;
+       struct iovec iovstack[UIO_FASTIOV], *iov = iovstack;
+       struct sockaddr_storage address;
+       struct msghdr msg = { .msg_name = &address };
+       struct sockaddr __user *uaddr;
+       ssize_t err;
 
-       return ___sys_recvmsg(sock, msg, &msg_sys, flags, 0);
+       err = recvmsg_copy_msghdr(&msg, umsg, flags, &uaddr, &iov);
+       if (err)
+               return err;
+       /* disallow ancillary data requests from this path */
+       if (msg.msg_control || msg.msg_controllen) {
+               err = -EINVAL;
+               goto out;
+       }
+
+       err = ____sys_recvmsg(sock, &msg, umsg, uaddr, flags, 0);
+out:
+       kfree(iov);
+       return err;
 }
 
 long __sys_recvmsg(int fd, struct user_msghdr __user *msg, unsigned int flags,
@@ -3161,6 +3230,8 @@ static int compat_ifr_data_ioctl(struct net *net, unsigned int cmd,
        struct ifreq ifreq;
        u32 data32;
 
+       if (!is_socket_ioctl_cmd(cmd))
+               return -ENOTTY;
        if (copy_from_user(ifreq.ifr_name, u_ifreq32->ifr_name, IFNAMSIZ))
                return -EFAULT;
        if (get_user(data32, &u_ifreq32->ifr_data))
@@ -3452,6 +3523,7 @@ static int compat_sock_ioctl_trans(struct file *file, struct socket *sock,
        case SIOCSARP:
        case SIOCGARP:
        case SIOCDARP:
+       case SIOCOUTQNSD:
        case SIOCATMARK:
                return sock_do_ioctl(net, sock, cmd, arg);
        }