Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/netbsd-6-0]: src/sys/compat Pull up following revision(s) (requested by ...



details:   https://anonhg.NetBSD.org/src/rev/56d57a0df9d7
branches:  netbsd-6-0
changeset: 775221:56d57a0df9d7
user:      snj <snj%NetBSD.org@localhost>
date:      Sat Aug 12 16:20:59 2017 +0000

description:
Pull up following revision(s) (requested by mrg in ticket #1469):
        sys/compat/common/vfs_syscalls_12.c: revision 1.30
        sys/compat/common/vfs_syscalls_43.c: revision 1.56
        sys/compat/sys/dirent.h: revision 1.3
It is wishful thinking that vn_readdir will return dirent12 structures.
--
Fix the compat-4.3 getdirentries call (pre d_type). This is used in NetBSD-0.9.
--
add a struct for the 4.3BSD struct direct

diffstat:

 sys/compat/common/vfs_syscalls_12.c |  137 ++++++++++++++++++++++--
 sys/compat/common/vfs_syscalls_43.c |  204 ++++++++++++++++++++---------------
 sys/compat/sys/dirent.h             |    9 +-
 3 files changed, 249 insertions(+), 101 deletions(-)

diffs (truncated from 452 to 300 lines):

diff -r b3a1af583553 -r 56d57a0df9d7 sys/compat/common/vfs_syscalls_12.c
--- a/sys/compat/common/vfs_syscalls_12.c       Thu Aug 10 07:04:19 2017 +0000
+++ b/sys/compat/common/vfs_syscalls_12.c       Sat Aug 12 16:20:59 2017 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: vfs_syscalls_12.c,v 1.29 2011/01/19 10:21:16 tsutsui Exp $     */
+/*     $NetBSD: vfs_syscalls_12.c,v 1.29.18.1 2017/08/12 16:20:59 snj Exp $    */
 
 /*
  * Copyright (c) 1989, 1993
@@ -37,7 +37,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls_12.c,v 1.29 2011/01/19 10:21:16 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls_12.c,v 1.29.18.1 2017/08/12 16:20:59 snj Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -56,6 +56,7 @@
 #include <sys/syscallargs.h>
 
 #include <compat/sys/stat.h>
+#include <compat/sys/dirent.h>
 
 /*
  * Convert from a new to an old stat structure.
@@ -96,28 +97,140 @@
                syscallarg(u_int) count;
                syscallarg(long *) basep;
        } */
+       struct dirent *bdp;
+       struct vnode *vp;
+       char *inp, *tbuf;               /* Current-format */
+       int len, reclen;                /* Current-format */
+       char *outp;                     /* Dirent12-format */
+       int resid, old_reclen = 0;      /* Dirent12-format */
        struct file *fp;
-       int error, done;
+       struct uio auio;
+       struct iovec aiov;
+       struct dirent12 idb;
+       off_t off;              /* true file offset */
+       int buflen, error, eofflag, nbytes;
+       struct vattr va;
+       off_t *cookiebuf = NULL, *cookie;
+       int ncookies;
        long loff;
-
+                
        /* fd_getvnode() will use the descriptor for us */
        if ((error = fd_getvnode(SCARG(uap, fd), &fp)) != 0)
-               return error;
+               return (error);
+
        if ((fp->f_flag & FREAD) == 0) {
                error = EBADF;
-               goto out;
+               goto out1;
        }
 
+       vp = (struct vnode *)fp->f_data;
+       if (vp->v_type != VDIR) {
+               error = ENOTDIR;
+               goto out1;
+       }
+
+       vn_lock(vp, LK_SHARED | LK_RETRY);
+       error = VOP_GETATTR(vp, &va, l->l_cred);
+       VOP_UNLOCK(vp);
+       if (error)
+               goto out1;
+
        loff = fp->f_offset;
+       nbytes = SCARG(uap, count);
+       buflen = min(MAXBSIZE, nbytes);
+       if (buflen < va.va_blocksize)
+               buflen = va.va_blocksize;
+       tbuf = malloc(buflen, M_TEMP, M_WAITOK);
 
-       error = vn_readdir(fp, SCARG(uap, buf), UIO_USERSPACE,
-                       SCARG(uap, count), &done, l, 0, 0);
+       vn_lock(vp, LK_EXCLUSIVE | LK_RETRY);
+       off = fp->f_offset;
+again:
+       aiov.iov_base = tbuf;
+       aiov.iov_len = buflen;
+       auio.uio_iov = &aiov;
+       auio.uio_iovcnt = 1;
+       auio.uio_rw = UIO_READ;
+       auio.uio_resid = buflen;
+       auio.uio_offset = off;
+       UIO_SETUP_SYSSPACE(&auio);
+       /*
+         * First we read into the malloc'ed buffer, then
+         * we massage it into user space, one record at a time.
+         */
+       error = VOP_READDIR(vp, &auio, fp->f_cred, &eofflag, &cookiebuf,
+           &ncookies);
+       if (error)
+               goto out;
+
+       inp = tbuf;
+       outp = SCARG(uap, buf);
+       resid = nbytes;
+       if ((len = buflen - auio.uio_resid) == 0)
+               goto eof;
 
-       error = copyout(&loff, SCARG(uap, basep), sizeof(long));
-       *retval = done;
- out:
+       for (cookie = cookiebuf; len > 0; len -= reclen) {
+               bdp = (struct dirent *)inp;
+               reclen = bdp->d_reclen;
+               if (reclen & 3)
+                       panic(__func__);
+               if (bdp->d_fileno == 0) {
+                       inp += reclen;  /* it is a hole; squish it out */
+                       if (cookie)
+                               off = *cookie++;
+                       else
+                               off += reclen;
+                       continue;
+               }
+               old_reclen = _DIRENT_RECLEN(&idb, bdp->d_namlen);
+               if (reclen > len || resid < old_reclen) {
+                       /* entry too big for buffer, so just stop */
+                       outp++;
+                       break;
+               }
+               /*
+                * Massage in place to make a Dirent12-shaped dirent (otherwise
+                * we have to worry about touching user memory outside of
+                * the copyout() call).
+                */
+               idb.d_fileno = (uint32_t)bdp->d_fileno;
+               idb.d_reclen = (uint16_t)old_reclen;
+               idb.d_type = (uint8_t)bdp->d_type;
+               idb.d_namlen = (uint8_t)bdp->d_namlen;
+               strcpy(idb.d_name, bdp->d_name);
+               if ((error = copyout(&idb, outp, old_reclen)))
+                       goto out;
+               /* advance past this real entry */
+               inp += reclen;
+               if (cookie)
+                       off = *cookie++; /* each entry points to itself */
+               else
+                       off += reclen;
+               /* advance output past Dirent12-shaped entry */
+               outp += old_reclen;
+               resid -= old_reclen;
+       }
+
+       /* if we squished out the whole block, try again */
+       if (outp == SCARG(uap, buf)) {
+               if (cookiebuf)
+                       free(cookiebuf, M_TEMP);
+               cookiebuf = NULL;
+               goto again;
+       }
+       fp->f_offset = off;     /* update the vnode offset */
+
+eof:
+       *retval = nbytes - resid;
+out:
+       VOP_UNLOCK(vp);
+       if (cookiebuf)
+               free(cookiebuf, M_TEMP);
+       free(tbuf, M_TEMP);
+out1:
        fd_putfile(SCARG(uap, fd));
-       return error;
+       if (error)
+               return error;
+       return copyout(&loff, SCARG(uap, basep), sizeof(long));
 }
 
 /*
diff -r b3a1af583553 -r 56d57a0df9d7 sys/compat/common/vfs_syscalls_43.c
--- a/sys/compat/common/vfs_syscalls_43.c       Thu Aug 10 07:04:19 2017 +0000
+++ b/sys/compat/common/vfs_syscalls_43.c       Sat Aug 12 16:20:59 2017 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: vfs_syscalls_43.c,v 1.54.20.1 2016/08/27 14:51:50 bouyer Exp $ */
+/*     $NetBSD: vfs_syscalls_43.c,v 1.54.20.2 2017/08/12 16:20:59 snj Exp $    */
 
 /*
  * Copyright (c) 1989, 1993
@@ -37,7 +37,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls_43.c,v 1.54.20.1 2016/08/27 14:51:50 bouyer Exp $");
+__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls_43.c,v 1.54.20.2 2017/08/12 16:20:59 snj Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_compat_netbsd.h"
@@ -69,6 +69,7 @@
 
 #include <compat/sys/stat.h>
 #include <compat/sys/mount.h>
+#include <compat/sys/dirent.h>
 
 #include <compat/common/compat_util.h>
 
@@ -373,112 +374,139 @@
                syscallarg(u_int) count;
                syscallarg(long *) basep;
        } */
+       struct dirent *bdp;
        struct vnode *vp;
+       char *inp, *tbuf;               /* Current-format */
+       int len, reclen;                /* Current-format */
+       char *outp;                     /* Dirent12-format */
+       int resid, old_reclen = 0;      /* Dirent12-format */
        struct file *fp;
-       struct uio auio, kuio;
-       struct iovec aiov, kiov;
-       struct dirent *dp, *edp;
-       char *dirbuf;
-       size_t count = min(MAXBSIZE, (size_t)SCARG(uap, count));
-
-       int error, eofflag, readcnt;
+       struct uio auio;
+       struct iovec aiov;
+       struct dirent43 idb;
+       off_t off;              /* true file offset */
+       int buflen, error, eofflag, nbytes;
+       struct vattr va;
+       off_t *cookiebuf = NULL, *cookie;
+       int ncookies;
        long loff;
-
+                
        /* fd_getvnode() will use the descriptor for us */
        if ((error = fd_getvnode(SCARG(uap, fd), &fp)) != 0)
                return (error);
+
        if ((fp->f_flag & FREAD) == 0) {
                error = EBADF;
-               goto out;
+               goto out1;
        }
+
        vp = (struct vnode *)fp->f_data;
-unionread:
        if (vp->v_type != VDIR) {
-               error = EINVAL;
-               goto out;
+               error = ENOTDIR;
+               goto out1;
        }
-       aiov.iov_base = SCARG(uap, buf);
-       aiov.iov_len = count;
+
+       vn_lock(vp, LK_SHARED | LK_RETRY);
+       error = VOP_GETATTR(vp, &va, l->l_cred);
+       VOP_UNLOCK(vp);
+       if (error)
+               goto out1;
+
+       loff = fp->f_offset;
+       nbytes = SCARG(uap, count);
+       buflen = min(MAXBSIZE, nbytes);
+       if (buflen < va.va_blocksize)
+               buflen = va.va_blocksize;
+       tbuf = malloc(buflen, M_TEMP, M_WAITOK);
+
+       vn_lock(vp, LK_EXCLUSIVE | LK_RETRY);
+       off = fp->f_offset;
+again:
+       aiov.iov_base = tbuf;
+       aiov.iov_len = buflen;
        auio.uio_iov = &aiov;
        auio.uio_iovcnt = 1;
        auio.uio_rw = UIO_READ;
-       auio.uio_resid = count;
-       KASSERT(l == curlwp);
-       auio.uio_vmspace = curproc->p_vmspace;
-       vn_lock(vp, LK_EXCLUSIVE | LK_RETRY);
-       loff = auio.uio_offset = fp->f_offset;
-#      if (BYTE_ORDER != LITTLE_ENDIAN)
-               if ((vp->v_mount->mnt_iflag & IMNT_DTYPE) == 0) {
-                       error = VOP_READDIR(vp, &auio, fp->f_cred, &eofflag,
-                           (off_t **)0, (int *)0);
-                       fp->f_offset = auio.uio_offset;
-               } else
-#      endif
-       {
-               kuio = auio;
-               kuio.uio_iov = &kiov;
-               kiov.iov_len = count;
-               dirbuf = malloc(count, M_TEMP, M_WAITOK);
-               kiov.iov_base = dirbuf;
-               UIO_SETUP_SYSSPACE(&kuio);
-               error = VOP_READDIR(vp, &kuio, fp->f_cred, &eofflag,
-                           (off_t **)0, (int *)0);
-               fp->f_offset = kuio.uio_offset;
-               if (error == 0) {
-                       readcnt = count - kuio.uio_resid;
-                       edp = (struct dirent *)&dirbuf[readcnt];
-                       for (dp = (struct dirent *)dirbuf; dp < edp; ) {



Home | Main Index | Thread Index | Old Index