From 52fe32278aee9f359d4ef6d1fab7be405ca0b193 Mon Sep 17 00:00:00 2001 From: Chuck Lever Date: Tue, 15 Jul 2008 11:59:03 -0400 Subject: [PATCH] The text-based mount command displays the rather inexplicable "mount: internal error" whenever it encounters a problem that is entirely unexpected by its designers. Let's beef that error message up to include instructions about reporting the problem, and fix the error code returned by the mount option rewriting logic so that also will no longer report "internal error". An error in there should generally only occur if there was an invalid mount option specified. Signed-off-by: Chuck Lever Signed-off-by: Steve Dickson --- utils/mount/error.c | 7 ++++++- utils/mount/stropts.c | 7 ++++--- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/utils/mount/error.c b/utils/mount/error.c index 147e919..5fd5705 100644 --- a/utils/mount/error.c +++ b/utils/mount/error.c @@ -231,8 +231,13 @@ void mount_error(const char *spec, const char *mount_point, int error) rpc_mount_errors((char *)spec, 0, 0); break; case EIO: + nfs_error(_("%s: mount system call failed"), progname); + break; case EFAULT: - nfs_error(_("%s: internal error"), progname); + nfs_error(_("%s: encountered unexpected error condition."), + progname); + nfs_error(_("%s: please report the error to" PACKAGE_BUGREPORT), + progname); break; default: nfs_error(_("%s: %s"), diff --git a/utils/mount/stropts.c b/utils/mount/stropts.c index 967fd69..c8148c4 100644 --- a/utils/mount/stropts.c +++ b/utils/mount/stropts.c @@ -356,12 +356,13 @@ static struct mount_options *rewrite_mount_options(char *str) clnt_addr_t nfs_server = { }; int p; - errno = EIO; - options = po_split(str); - if (!options) + if (!options) { + errno = EFAULT; return NULL; + } + errno = EINVAL; option = po_get(options, "addr"); if (option) { nfs_server.saddr.sin_family = AF_INET; -- 2.39.5