Applied patch from Matt Kraai to add v*error functions for handling vararg

lists in a safe 'n sane fashion.

1 files changed, 21 insertions, 21 deletions

diff --git a/utility.c b/utility.c
index 7c41ab8..eb33383 100644
--- a/utility.c
+++ b/utility.c
@@ -84,36 +84,35 @@ extern void usage(const char *usage)
 	exit(EXIT_FAILURE);
 }
 
-extern void errorMsg(const char *s, ...)
+static void verrorMsg(const char *s, va_list p)
 {
-	va_list p;
-
-	va_start(p, s);
 	fflush(stdout);
 	fprintf(stderr, "%s: ", applet_name);
 	vfprintf(stderr, s, p);
-	va_end(p);
 	fflush(stderr);
 }
 
-extern void fatalError(const char *s, ...)
+extern void errorMsg(const char *s, ...)
 {
 	va_list p;
 
 	va_start(p, s);
-	fflush(stdout);
-	fprintf(stderr, "%s: ", applet_name);
-	vfprintf(stderr, s, p);
+	verrorMsg(s, p);
 	va_end(p);
-	fflush(stderr);
-	exit(EXIT_FAILURE);
 }
 
-extern void perrorMsg(const char *s, ...)
+extern void fatalError(const char *s, ...)
 {
 	va_list p;
 
 	va_start(p, s);
+	verrorMsg(s, p);
+	va_end(p);
+	exit(EXIT_FAILURE);
+}
+
+static void vperrorMsg(const char *s, va_list p)
+{
 	fflush(stdout);
 	fprintf(stderr, "%s: ", applet_name);
 	if (s && *s) {
@@ -121,24 +120,25 @@ extern void perrorMsg(const char *s, ...)
 		fputs(": ", stderr);
 	}
 	fprintf(stderr, "%s\n", strerror(errno));
-	va_end(p);
 	fflush(stderr);
 }
 
+extern void perrorMsg(const char *s, ...)
+{
+	va_list p;
+
+	va_start(p, s);
+	vperrorMsg(s, p);
+	va_end(p);
+}
+
 extern void fatalPerror(const char *s, ...)
 {
 	va_list p;
 
 	va_start(p, s);
-	fflush(stdout);
-	fprintf(stderr, "%s: ", applet_name);
-	if (s && *s) {
-		vfprintf(stderr, s, p);
-		fputs(": ", stderr);
-	}
-	fprintf(stderr, "%s\n", strerror(errno));
+	vperrorMsg(s, p);
 	va_end(p);
-	fflush(stderr);
 	exit(EXIT_FAILURE);
 }