diff options
| author | adrian <adrian@FreeBSD.org> | 2001-03-01 21:00:17 +0000 |
|---|---|---|
| committer | adrian <adrian@FreeBSD.org> | 2001-03-01 21:00:17 +0000 |
| commit | 401895533496c6f7943c0cadf487c2cfa8f852ac (patch) | |
| tree | 19b63a29b1f18d2efdee5765c5bffd371838dd1e /sys/ntfs | |
| parent | 845eeac4fc47466824d871b49fc54af4c3812e09 (diff) | |
| download | FreeBSD-src-401895533496c6f7943c0cadf487c2cfa8f852ac.zip FreeBSD-src-401895533496c6f7943c0cadf487c2cfa8f852ac.tar.gz | |
Reviewed by: jlemon
An initial tidyup of the mount() syscall and VFS mount code.
This code replaces the earlier work done by jlemon in an attempt to
make linux_mount() work.
* the guts of the mount work has been moved into vfs_mount().
* move `type', `path' and `flags' from being userland variables into being
kernel variables in vfs_mount(). `data' remains a pointer into
userspace.
* Attempt to verify the `type' and `path' strings passed to vfs_mount()
aren't too long.
* rework mount() and linux_mount() to take the userland parameters
(besides data, as mentioned) and pass kernel variables to vfs_mount().
(linux_mount() already did this, I've just tidied it up a little more.)
* remove the copyin*() stuff for `path'. `data' still requires copyin*()
since its a pointer into userland.
* set `mount->mnt_statf_mntonname' in vfs_mount() rather than in each
filesystem. This variable is generally initialised with `path', and
each filesystem can override it if they want to.
* NOTE: f_mntonname is intiailised with "/" in the case of a root mount.
Diffstat (limited to 'sys/ntfs')
| -rw-r--r-- | sys/ntfs/ntfs_vfsops.c | 14 |
1 files changed, 5 insertions, 9 deletions
diff --git a/sys/ntfs/ntfs_vfsops.c b/sys/ntfs/ntfs_vfsops.c index 384883e..89a7dd7 100644 --- a/sys/ntfs/ntfs_vfsops.c +++ b/sys/ntfs/ntfs_vfsops.c @@ -376,16 +376,12 @@ ntfs_mount ( /* * Since this is a new mount, we want the names for * the device and the mount point copied in. If an - * error occurs, the mountpoint is discarded by the - * upper level code. + * error occurs, the mountpoint is discarded by the + * upper level code. Note that vfs_mount() handles + * copying the mountpoint f_mntonname for us, so we + * don't have to do it here unless we want to set it + * to something other than "path" for some rason. */ - /* Save "last mounted on" info for mount point (NULL pad)*/ - copyinstr( path, /* mount point*/ - mp->mnt_stat.f_mntonname, /* save area*/ - MNAMELEN - 1, /* max size*/ - &size); /* real size*/ - bzero( mp->mnt_stat.f_mntonname + size, MNAMELEN - size); - /* Save "mounted from" info for mount point (NULL pad)*/ copyinstr( args.fspec, /* device name*/ mp->mnt_stat.f_mntfromname, /* save area*/ |
