X-Git-Url: https://git.ucc.asn.au/?a=blobdiff_plain;f=server%2Flog.c;h=d41021f43032d3fe335497a402a494417eaef242;hb=787ccba446c684d86ecf598de985eddbfa18652f;hp=6f44bb0e05789c6c66a3164be040af75166f49dd;hpb=43c1519cb60f8fef09043b9af6f43d319db12e3d;p=matches%2FMCTX3420.git

diff --git a/server/log.c b/server/log.c
index 6f44bb0..d41021f 100644
--- a/server/log.c
+++ b/server/log.c
@@ -1,22 +1,23 @@
 /**
  * @file log.c
- * @purpose Implement logging and error handling functions
+ * @brief Implement logging and error handling functions
  */
 
 
-#include <unistd.h>
-
 // --- Custom headers --- //
+#include "common.h"
 #include "log.h"
 #include "options.h"
 
-// --- Static variables --- //
-static const char * unspecified_funct = "???";
+#include <unistd.h>
+#include <syslog.h>
+#include <stdarg.h>
 
-// --- Function implementations --- //
+static const char * unspecified_funct = "???";
 
 /**
- * Print a message to stderr
+ * Print a message to stderr and log it via syslog. The message must be
+ * less than BUFSIZ characters long, or it will be truncated.
  * @param level - Specify how severe the message is.
 	If level is higher (less urgent) than the program's verbosity (see options.h) no message will be printed
  * @param funct - String indicating the function name from which this function was called.
@@ -26,17 +27,23 @@ static const char * unspecified_funct = "???";
  */
 void LogEx(int level, const char * funct, ...)
 {
+	//Todo: consider setlogmask(3) to filter messages
 	const char *fmt;
+	char buffer[BUFSIZ];
 	va_list va;
+
+	// Don't print the message unless we need to
+	if (level > g_options.verbosity)
+		return;
+
 	va_start(va, funct);
 	fmt = va_arg(va, const char*);
 	
 	if (fmt == NULL) // sanity check
-		FatalEx("Log", "Format string is NULL");
+		Fatal("Format string is NULL");
 
-	// Don't print the message unless we need to
-	if (level > g_options.verbosity)
-		return;
+	vsnprintf(buffer, BUFSIZ, fmt, va);
+	va_end(va);
 
 	if (funct == NULL)
 		funct = unspecified_funct;
@@ -46,31 +53,28 @@ void LogEx(int level, const char * funct, ...)
 	switch (level)
 	{
 		case LOGERR:
+			level = LOG_ERR;
 			severity = "ERROR";
 			break;
 		case LOGWARN:
+			level = LOG_WARNING;
 			severity = "WARNING";
 			break;
 		case LOGNOTE:
+			level = LOG_NOTICE;
 			severity = "NOTICE";
 			break;
 		case LOGINFO:
+			level = LOG_INFO;
 			severity = "INFO";
 			break;
 		default:
+			level = LOG_DEBUG;
 			severity = "DEBUG";
 			break;
 	}
 
-	// Print: Program name, PID, severity string, function name first
-	fprintf(stderr, "%s [%d] : %s : %s - ", g_options.program, getpid(), severity, funct);
-
-	// Then pass additional arguments with the format string to vfprintf for printing
-	vfprintf(stderr, fmt, va);
-	va_end(va);
-
-	// End log messages with a newline
-	fprintf(stderr, "\n");
+	syslog(level, "%s: %s - %s", severity, funct, buffer);
 }
 
 /**
@@ -83,6 +87,7 @@ void LogEx(int level, const char * funct, ...)
 void FatalEx(const char * funct, ...)
 {
 	const char *fmt;
+	char buffer[BUFSIZ];
 	va_list va;
 	va_start(va, funct);
 	fmt = va_arg(va, const char*);
@@ -91,19 +96,17 @@ void FatalEx(const char * funct, ...)
 	{
 		// Fatal error in the Fatal function.
 		// (This really shouldn't happen unless someone does something insanely stupid)
-		FatalEx("Fatal", "Format string is NULL");
+		Fatal("Format string is NULL");
 		return; // Should never get here
 	}
 
+	vsnprintf(buffer, BUFSIZ, fmt,va);
+	va_end(va);
+
 	if (funct == NULL)
 		funct = unspecified_funct;
 
-	fprintf(stderr, "%s [%d] : %s : FATAL - ", g_options.program, getpid(), funct);
-
-	vfprintf(stderr, fmt, va);
-	va_end(va);
-	fprintf(stderr, "\n");
-
+	syslog(LOG_CRIT, "FATAL: %s - %s", funct, buffer);
 	exit(EXIT_FAILURE);
 }