diff options
author | jkh <jkh@FreeBSD.org> | 1996-03-03 08:44:22 +0000 |
---|---|---|
committer | jkh <jkh@FreeBSD.org> | 1996-03-03 08:44:22 +0000 |
commit | a4107fca32bc1c4b153c0940afcc26f8c6b6b742 (patch) | |
tree | 71c986fdb0469c008960c5488b5bd0b8340fb63e /sbin/mount_ifs | |
parent | 6195d71f7ca9da292bed0d3c5cfcb5a8a5c5e87d (diff) | |
download | FreeBSD-src-a4107fca32bc1c4b153c0940afcc26f8c6b6b742.zip FreeBSD-src-a4107fca32bc1c4b153c0940afcc26f8c6b6b742.tar.gz |
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.
Diffstat (limited to 'sbin/mount_ifs')
-rw-r--r-- | sbin/mount_ifs/mount.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/sbin/mount_ifs/mount.c b/sbin/mount_ifs/mount.c index cf05d3f..1be00ec 100644 --- a/sbin/mount_ifs/mount.c +++ b/sbin/mount_ifs/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. */ |