X-Git-Url: http://git.code-monkey.de/?a=blobdiff_plain;f=src%2Flog.c;h=91ec5b023ed40e0de5d8e407553114fc0f9d78f7;hb=21ed5998124647e79de379c35b75acaf7323677a;hp=ad05490c7232b900df69dcbc7c971955a645670f;hpb=134ce814d444d90c229d0b1777d575bc1effa718;p=umurmur.git

diff --git a/src/log.c b/src/log.c
index ad05490..91ec5b0 100644
--- a/src/log.c
+++ b/src/log.c
@@ -1,5 +1,5 @@
-/* Copyright (C) 2009-2013, Martin Johansson <martin@fatbob.nu>
-   Copyright (C) 2005-2013, Thorvald Natvig <thorvald@natvig.com>
+/* Copyright (C) 2009-2014, Martin Johansson <martin@fatbob.nu>
+   Copyright (C) 2005-2014, Thorvald Natvig <thorvald@natvig.com>
 
    All rights reserved.
 
@@ -39,6 +39,7 @@
 
 #include "log.h"
 #include "conf.h"
+#include "util.h"
 
 #define STRSIZE 254
 
@@ -56,7 +57,7 @@ static void openlogfile(const char *logfilename)
 	/* Set the stream as line buffered */
 	if (setvbuf(logfile, NULL, _IOLBF, 0) < 0)
 		Log_fatal("setvbuf() failed: %s\n", strerror(errno));
-	
+
 	/* XXX - Is it neccessary/appropriate that logging to file is non-blocking?
 	 * If not, there's a risk that execution blocks, meaning that voice blocks
 	 * as well since uMurmur is single threaded by design. OTOH, what could
@@ -84,11 +85,11 @@ static char *timestring(void)
 void Log_init(bool_t terminal)
 {
 	const char *logfilename;
-	
-	termprint = terminal;		
+
+	termprint = terminal;
 	if (termprint)
 		return;
-	
+
 	logfilename = getStrConf(LOGFILE);
 	if (logfilename != NULL) {
 		openlogfile(logfilename);
@@ -103,14 +104,14 @@ void Log_free()
 		return;
 	else if (logfile)
 		fclose(logfile);
-	else 
+	else
 		closelog();
 }
-		
+
 void Log_reset()
 {
 	const char *logfilename;
-	
+
 	if (logfile) {
 		logfilename = getStrConf(LOGFILE);
 		fclose(logfile);
@@ -122,11 +123,11 @@ void logthis(const char *logstring, ...)
 {
 	va_list argp;
 	char buf[STRSIZE + 1];
-	
+
 	va_start(argp, logstring);
 	vsnprintf(&buf[0], STRSIZE, logstring, argp);
 	va_end(argp);
-	
+
 	if (termprint)
 		fprintf(stderr, "%s\n", buf);
 	else if (logfile)
@@ -140,14 +141,14 @@ void Log_warn(const char *logstring, ...)
 	va_list argp;
 	char buf[STRSIZE + 1];
 	int offset = 0;
-	
+
 	if (termprint || logfile)
 		offset = sprintf(buf, "WARN: ");
-	
+
 	va_start(argp, logstring);
 	vsnprintf(&buf[offset], STRSIZE - offset, logstring, argp);
 	va_end(argp);
-	
+
 	if (termprint)
 		fprintf(stderr, "%s\n", buf);
 	else if (logfile)
@@ -161,14 +162,14 @@ void Log_info(const char *logstring, ...)
 	va_list argp;
 	char buf[STRSIZE + 1];
 	int offset = 0;
-	
+
 	if (termprint || logfile)
 		offset = sprintf(buf, "INFO: ");
-	
-	va_start(argp, logstring);	
+
+	va_start(argp, logstring);
 	vsnprintf(&buf[offset], STRSIZE - offset, logstring, argp);
 	va_end(argp);
-	
+
 	if (termprint)
 		fprintf(stderr, "%s\n", buf);
 	else if (logfile)
@@ -182,19 +183,22 @@ void Log_info_client(client_t *client, const char *logstring, ...)
 	va_list argp;
 	char buf[STRSIZE + 1];
 	int offset = 0;
-	
+
 	if (termprint || logfile)
 		offset = sprintf(buf, "INFO: ");
 
 	va_start(argp, logstring);
 	offset += vsnprintf(&buf[offset], STRSIZE - offset, logstring, argp);
 	va_end(argp);
-	
+
+	char *clientAddressString = Util_clientAddressToString(client);
 	offset += snprintf(&buf[offset], STRSIZE - offset, " - [%d] %s@%s:%d",
-					   client->sessionId,
-					   client->username == NULL ? "" : client->username,
-					   inet_ntoa(client->remote_tcp.sin_addr),
-					   ntohs(client->remote_tcp.sin_port));
+		client->sessionId,
+		client->username == NULL ? "" : client->username,
+		clientAddressString,
+		Util_clientAddressToPortTCP(client));
+	free(clientAddressString);
+
 	if (termprint)
 		fprintf(stderr, "%s\n", buf);
 	else if (logfile)
@@ -209,11 +213,11 @@ void Log_debug(const char *logstring, ...)
 	va_list argp;
 	char buf[STRSIZE + 1];
 	int offset = 0;
-	
+
 	if (termprint || logfile)
 		offset = sprintf(buf, "DEBUG: ");
-	
-	va_start(argp, logstring);	
+
+	va_start(argp, logstring);
 	vsnprintf(&buf[offset], STRSIZE - offset, logstring, argp);
 	va_end(argp);
 	if (termprint)
@@ -230,20 +234,20 @@ void Log_fatal(const char *logstring, ...)
 	va_list argp;
 	char buf[STRSIZE + 1];
 	int offset = 0;
-	
+
 	if (termprint || logfile)
 		offset = sprintf(buf, "FATAL: ");
-	
-	va_start(argp, logstring);	
+
+	va_start(argp, logstring);
 	vsnprintf(&buf[offset], STRSIZE - offset, logstring, argp);
 	va_end(argp);
-	
+
 	if (termprint)
 		fprintf(stderr, "%s\n", buf);
 	else if (logfile)
 		fprintf(logfile, "%s %s\n", timestring(), buf);
 	else { /* If logging subsystem is not initialized, fall back to stderr +
-			* syslog logging for fatal errors. 
+			* syslog logging for fatal errors.
 			*/
 		if (!init) {
 			openlog("uMurmurd", LOG_PID, LOG_DAEMON);
@@ -251,6 +255,6 @@ void Log_fatal(const char *logstring, ...)
 		}
 		syslog(LOG_CRIT, "%s", buf);
 	}
-	
+
 	exit(1);
 }