diff options
author | Jordan K. Hubbard <jkh@FreeBSD.org> | 1996-03-03 08:44:22 +0000 |
---|---|---|
committer | Jordan K. Hubbard <jkh@FreeBSD.org> | 1996-03-03 08:44:22 +0000 |
commit | 66fa281751b8ea67420c912faddbd189d426918e (patch) | |
tree | 71c986fdb0469c008960c5488b5bd0b8340fb63e /sbin/mount | |
parent | af6646cdc5304ff371facb2595939a4dfcbb23ab (diff) | |
download | src-66fa281751b8ea67420c912faddbd189d426918e.tar.gz src-66fa281751b8ea67420c912faddbd189d426918e.zip |
Close PR#17. This may be a contraversal fix in that now mount will
spit out two error lines for a bogus filesystem type, e.g:
root@time-> mount -t foo /dev/sd0a /mnt
mount: exec /sbin/mount_foo for /mnt: No such file or directory
mount: exec /usr/sbin/mount_foo for /mnt: No such file or directory
But I would submit that if you're even going to scan multiple directories
for a mount_foo (which I actually think is somewhat bogus - if it's not
in /sbin, you're probably in big trouble anyway), you should emit an error
for each one. I got multiple complaints (in addition to the PR) that the
existing behavior was very confusing.
Notes
Notes:
svn path=/head/; revision=14350
Diffstat (limited to 'sbin/mount')
-rw-r--r-- | sbin/mount/mount.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/sbin/mount/mount.c b/sbin/mount/mount.c index cf05d3fa4a23..1be00ec7d26c 100644 --- a/sbin/mount/mount.c +++ b/sbin/mount/mount.c @@ -332,12 +332,8 @@ mountfs(vfstype, spec, name, flags, options, mntopts) (void)snprintf(execname, sizeof(execname), "%s/mount_%s", *edir, vfstype); execv(execname, (char * const *)argv); - if (errno != ENOENT) - warn("exec %s for %s", execname, name); - } while (*++edir != NULL); - - if (errno == ENOENT) warn("exec %s for %s", execname, name); + } while (*++edir != NULL); exit(1); /* NOTREACHED */ default: /* Parent. */ |