aboutsummaryrefslogtreecommitdiff
path: root/sbin/mount
diff options
context:
space:
mode:
authorJordan K. Hubbard <jkh@FreeBSD.org>1996-03-03 08:44:22 +0000
committerJordan K. Hubbard <jkh@FreeBSD.org>1996-03-03 08:44:22 +0000
commit66fa281751b8ea67420c912faddbd189d426918e (patch)
tree71c986fdb0469c008960c5488b5bd0b8340fb63e /sbin/mount
parentaf6646cdc5304ff371facb2595939a4dfcbb23ab (diff)
downloadsrc-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.c6
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. */