[Ocfs2-devel] [PATCH] ocfs2-tools: add error message to mount.ocfs2 when mount point is invalid.
Joel Becker
Joel.Becker at oracle.com
Thu Jul 31 13:00:56 PDT 2008
On Thu, Jul 31, 2008 at 09:40:21PM +0800, Coly Li wrote:
> diff --git a/mount.ocfs2/mount.ocfs2.c b/mount.ocfs2/mount.ocfs2.c
> index a97b4cb..eb8e336 100644
> --- a/mount.ocfs2/mount.ocfs2.c
> +++ b/mount.ocfs2/mount.ocfs2.c
> @@ -382,9 +382,21 @@ int main(int argc, char **argv)
> o2cb_complete_group_join(&cluster, &desc, errno);
> }
> block_signals (SIG_UNBLOCK);
> - com_err(progname, ret, "while mounting %s on %s. "
> - "Check 'dmesg' for more information on this error.",
> - mo.dev, mo.dir);
> +
> + /* complain mount failure */
> + if (lstat(mo.dir, &statbuf))
> + com_err(progname, 0, "mount point %s does not "
> + "exist", mo.dir);
> + else if (stat(mo.dir, &statbuf))
> + com_err(progname, 0, "mount point %s is a "
> + "symbolic link to nowhere", mo.dir);
> + else if (stat(mo.dir, &statbuf) || !S_ISDIR(statbuf.st_mode))
+ else if (!S_ISDIR(statbuf.st_mode))
You successfully called stat in the previous "else if" clause.
> + com_err(progname, 0, "mount point %s is not "
> + "a directory", mo.dir);
> + else
> + com_err(progname, ret, "while mounting %s on %s. "
> + "Check 'dmesg' for more information on this "
> + "error.", mo.dev, mo.dir);
Also, I agree with sunil's "broken symbolic link" text.
Joel
--
"I have never let my schooling interfere with my education."
- Mark Twain
Joel Becker
Principal Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127
More information about the Ocfs2-devel
mailing list