mirror of
https://git.hardenedbsd.org/hardenedbsd/HardenedBSD.git
synced 2024-11-16 15:44:04 +01:00
Backout my changes to disallow "mount /mnt /mnt" until I can verify
that nfs mounts work again (I locked up my home machine testing it and can't see what happened until I get home from work tonight).
This commit is contained in:
parent
def2f69e6f
commit
d5f6aa5f24
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=14566
@ -262,7 +262,6 @@ mountfs(vfstype, spec, name, flags, options, mntopts)
|
||||
pid_t pid;
|
||||
int argc, i, status;
|
||||
char *optbuf, execname[MAXPATHLEN + 1], mntpath[MAXPATHLEN];
|
||||
char specpath[MAXPATHLEN];
|
||||
|
||||
if (realpath(name, mntpath) != NULL && stat(mntpath, &sb) == 0) {
|
||||
if (!S_ISDIR(sb.st_mode)) {
|
||||
@ -274,27 +273,6 @@ mountfs(vfstype, spec, name, flags, options, mntopts)
|
||||
return (1);
|
||||
}
|
||||
|
||||
/*
|
||||
* The following check is a kludge to prevent the caller from
|
||||
* accidently using the file as the special device file and
|
||||
* the mount point. This will cause a panic due to a recursive
|
||||
* vnode lock. After some of the planned reworking of the
|
||||
* file system code is done, the kernel can be fixed properly
|
||||
* and this stupid check can be removed.
|
||||
*/
|
||||
if (realpath(spec, specpath) == NULL) {
|
||||
warn("%s", specpath);
|
||||
return (1);
|
||||
}
|
||||
if (strcmp(vfstype, "ufs") == 0 || strcmp(vfstype, "msdos") == 0 ||
|
||||
strcmp(vfstype, "cd9660") == 0) {
|
||||
if (strcmp(mntpath, specpath) == 0) {
|
||||
warnx("%s: Special device file and mount point may not be the same",
|
||||
specpath);
|
||||
return (1);
|
||||
}
|
||||
}
|
||||
|
||||
if (mntopts == NULL)
|
||||
mntopts = "";
|
||||
|
||||
|
@ -262,7 +262,6 @@ mountfs(vfstype, spec, name, flags, options, mntopts)
|
||||
pid_t pid;
|
||||
int argc, i, status;
|
||||
char *optbuf, execname[MAXPATHLEN + 1], mntpath[MAXPATHLEN];
|
||||
char specpath[MAXPATHLEN];
|
||||
|
||||
if (realpath(name, mntpath) != NULL && stat(mntpath, &sb) == 0) {
|
||||
if (!S_ISDIR(sb.st_mode)) {
|
||||
@ -274,27 +273,6 @@ mountfs(vfstype, spec, name, flags, options, mntopts)
|
||||
return (1);
|
||||
}
|
||||
|
||||
/*
|
||||
* The following check is a kludge to prevent the caller from
|
||||
* accidently using the file as the special device file and
|
||||
* the mount point. This will cause a panic due to a recursive
|
||||
* vnode lock. After some of the planned reworking of the
|
||||
* file system code is done, the kernel can be fixed properly
|
||||
* and this stupid check can be removed.
|
||||
*/
|
||||
if (realpath(spec, specpath) == NULL) {
|
||||
warn("%s", specpath);
|
||||
return (1);
|
||||
}
|
||||
if (strcmp(vfstype, "ufs") == 0 || strcmp(vfstype, "msdos") == 0 ||
|
||||
strcmp(vfstype, "cd9660") == 0) {
|
||||
if (strcmp(mntpath, specpath) == 0) {
|
||||
warnx("%s: Special device file and mount point may not be the same",
|
||||
specpath);
|
||||
return (1);
|
||||
}
|
||||
}
|
||||
|
||||
if (mntopts == NULL)
|
||||
mntopts = "";
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user