From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Price Date: Wed, 18 Jan 2012 16:39:31 +0000 Subject: [Cluster-devel] [PATCH 5/5] mkfs.gfs2: Improve error messages In-Reply-To: <1326904771-12416-1-git-send-email-anprice@redhat.com> References: <1326904771-12416-1-git-send-email-anprice@redhat.com> Message-ID: <1326904771-12416-5-git-send-email-anprice@redhat.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Currently mkfs.gfs2 does not warn that the lock table has been omitted, it just complains about a missing colon in the lock table. It also prefixes error messages with the absolute path of the source file. This patch removes the path from error messages (in mkfs and edit) and adds a check for an empty lock table. Signed-off-by: Andrew Price --- gfs2/edit/hexedit.h | 1 - gfs2/mkfs/gfs2_mkfs.h | 1 - gfs2/mkfs/main_mkfs.c | 4 ++++ 3 files changed, 4 insertions(+), 2 deletions(-) diff --git a/gfs2/edit/hexedit.h b/gfs2/edit/hexedit.h index b05b5c0..dd76810 100644 --- a/gfs2/edit/hexedit.h +++ b/gfs2/edit/hexedit.h @@ -129,7 +129,6 @@ static __inline__ __attribute__((noreturn, format (printf, 1, 2))) void die(const char *fmt, ...) { va_list ap; - fprintf(stderr, "%s: ", __FILE__); va_start(ap, fmt); vfprintf(stderr, fmt, ap); va_end(ap); diff --git a/gfs2/mkfs/gfs2_mkfs.h b/gfs2/mkfs/gfs2_mkfs.h index d7d7da7..231f370 100644 --- a/gfs2/mkfs/gfs2_mkfs.h +++ b/gfs2/mkfs/gfs2_mkfs.h @@ -24,7 +24,6 @@ static __inline__ __attribute__((noreturn, format (printf, 1, 2))) void die(const char *fmt, ...) { va_list ap; - fprintf(stderr, "%s: ", __FILE__); va_start(ap, fmt); vfprintf(stderr, fmt, ap); va_end(ap); diff --git a/gfs2/mkfs/main_mkfs.c b/gfs2/mkfs/main_mkfs.c index 3658fd4..7c93621 100644 --- a/gfs2/mkfs/main_mkfs.c +++ b/gfs2/mkfs/main_mkfs.c @@ -257,6 +257,10 @@ static void test_locking(char *lockproto, char *locktable) /* Nolock is always ok. */ } else if (strcmp(lockproto, "lock_gulm") == 0 || strcmp(lockproto, "lock_dlm") == 0) { + if (locktable == NULL || *locktable == '\0') { + fprintf(stderr, _("No lock table specified.\n")); + exit(-1); + } for (c = locktable; *c; c++) { if (isspace(*c)) die( _("locktable error: contains space characters\n")); -- 1.7.7.5