Blame SOURCES/autofs-5.1.2-log-functions-to-prefix-messages-with-attempt_id-if-available.patch

304803
autofs-5.1.2 - log functions to prefix messages with attempt_id if available
304803
304803
From: Lars R. Damerow <lars@pixar.com>
304803
304803
Now that there should be a mount attempt id try and include it in log entries.
304803
304803
Signed-off-by: Lars R. Damerow <lars@pixar.com>
304803
Signed-off-by: Ian Kent <raven@themaw.net>
304803
---
304803
 CHANGELOG |    1 
304803
 lib/log.c |  164 +++++++++++++++++++++++++++++++++++++++++++++++++++-----------
304803
 2 files changed, 137 insertions(+), 28 deletions(-)
304803
304803
--- autofs-5.0.7.orig/CHANGELOG
304803
+++ autofs-5.0.7/CHANGELOG
304803
@@ -237,6 +237,7 @@
304803
 - delay submount exit for amd submounts.
304803
 - add the mount requestor's pid to pending_args.
304803
 - create thread-local ID for mount attempts.
304803
+- log functions to prefix messages with attempt_id if available.
304803
 
304803
 25/07/2012 autofs-5.0.7
304803
 =======================
304803
--- autofs-5.0.7.orig/lib/log.c
304803
+++ autofs-5.0.7/lib/log.c
304803
@@ -32,6 +32,26 @@ static unsigned int logging_to_syslog =
304803
 static unsigned int do_verbose = 0;		/* Verbose feedback option */
304803
 static unsigned int do_debug = 0;		/* Full debug output */
304803
 
304803
+static char *prepare_attempt_prefix(const char *msg)
304803
+{
304803
+	unsigned long *attempt_id;
304803
+	char buffer[ATTEMPT_ID_SIZE + 1];
304803
+	char *prefixed_msg = NULL;
304803
+
304803
+	attempt_id = pthread_getspecific(key_thread_attempt_id);
304803
+	if (attempt_id) {
304803
+		int len = sizeof(buffer) + 1 + strlen(msg) + 1;
304803
+
304803
+		snprintf(buffer, ATTEMPT_ID_SIZE, "%02lx", *attempt_id);
304803
+		prefixed_msg = (char *) calloc(len, sizeof(char));
304803
+		strcpy(prefixed_msg, buffer);
304803
+		strcat(prefixed_msg, "|");
304803
+		strcat(prefixed_msg, msg);
304803
+	}
304803
+
304803
+	return prefixed_msg;
304803
+}
304803
+
304803
 void set_log_norm(void)
304803
 {
304803
 	do_verbose = 0;
304803
@@ -72,124 +92,212 @@ void set_log_debug_ap(struct autofs_poin
304803
 void log_info(unsigned int logopt, const char *msg, ...)
304803
 {
304803
 	unsigned int opt_log = logopt & (LOGOPT_DEBUG | LOGOPT_VERBOSE);
304803
+	char *prefixed_msg;
304803
 	va_list ap;
304803
 
304803
 	if (!do_debug && !do_verbose && !opt_log)
304803
 		return;
304803
 
304803
+	prefixed_msg = prepare_attempt_prefix(msg);
304803
+
304803
 	va_start(ap, msg);
304803
-	if (logging_to_syslog)
304803
-		vsyslog(LOG_INFO, msg, ap);
304803
-	else {
304803
-		vfprintf(stderr, msg, ap);
304803
+	if (logging_to_syslog) {
304803
+		if (prefixed_msg)
304803
+			vsyslog(LOG_INFO, prefixed_msg, ap);
304803
+		else
304803
+			vsyslog(LOG_INFO, msg, ap);
304803
+	} else {
304803
+		if (prefixed_msg)
304803
+			vfprintf(stderr, prefixed_msg, ap);
304803
+		else
304803
+			vfprintf(stderr, msg, ap);
304803
 		fputc('\n', stderr);
304803
 	}
304803
 	va_end(ap);
304803
 
304803
+	if (prefixed_msg)
304803
+		free(prefixed_msg);
304803
+
304803
 	return;
304803
 }
304803
 
304803
 void log_notice(unsigned int logopt, const char *msg, ...)
304803
 {
304803
 	unsigned int opt_log = logopt & (LOGOPT_DEBUG | LOGOPT_VERBOSE);
304803
+	char *prefixed_msg;
304803
 	va_list ap;
304803
 
304803
 	if (!do_debug && !do_verbose && !opt_log)
304803
 		return;
304803
 
304803
+	prefixed_msg = prepare_attempt_prefix(msg);
304803
+
304803
 	va_start(ap, msg);
304803
-	if (logging_to_syslog)
304803
-		vsyslog(LOG_NOTICE, msg, ap);
304803
-	else {
304803
-		vfprintf(stderr, msg, ap);
304803
+	if (logging_to_syslog) {
304803
+		if (prefixed_msg)
304803
+			vsyslog(LOG_NOTICE, prefixed_msg, ap);
304803
+		else
304803
+			vsyslog(LOG_INFO, msg, ap);
304803
+	} else {
304803
+		if (prefixed_msg)
304803
+			vfprintf(stderr, prefixed_msg, ap);
304803
+		else
304803
+			vfprintf(stderr, msg, ap);
304803
 		fputc('\n', stderr);
304803
 	}
304803
 	va_end(ap);
304803
 
304803
+	if (prefixed_msg)
304803
+		free(prefixed_msg);
304803
+
304803
 	return;
304803
 }
304803
 
304803
 void log_warn(unsigned int logopt, const char *msg, ...)
304803
 {
304803
 	unsigned int opt_log = logopt & (LOGOPT_DEBUG | LOGOPT_VERBOSE);
304803
+	char *prefixed_msg;
304803
 	va_list ap;
304803
 
304803
 	if (!do_debug && !do_verbose && !opt_log)
304803
 		return;
304803
 
304803
+	prefixed_msg = prepare_attempt_prefix(msg);
304803
+
304803
 	va_start(ap, msg);
304803
-	if (logging_to_syslog)
304803
-		vsyslog(LOG_WARNING, msg, ap);
304803
-	else {
304803
-		vfprintf(stderr, msg, ap);
304803
+	if (logging_to_syslog) {
304803
+		if (prefixed_msg)
304803
+			vsyslog(LOG_WARNING, prefixed_msg, ap);
304803
+		else
304803
+			vsyslog(LOG_INFO, msg, ap);
304803
+	} else {
304803
+		if (prefixed_msg)
304803
+			vfprintf(stderr, prefixed_msg, ap);
304803
+		else
304803
+			vfprintf(stderr, msg, ap);
304803
 		fputc('\n', stderr);
304803
 	}
304803
 	va_end(ap);
304803
 
304803
+	if (prefixed_msg)
304803
+		free(prefixed_msg);
304803
+
304803
 	return;
304803
 }
304803
 
304803
 void log_error(unsigned logopt, const char *msg, ...)
304803
 {
304803
+	char *prefixed_msg;
304803
 	va_list ap;
304803
 
304803
+	prefixed_msg = prepare_attempt_prefix(msg);
304803
+
304803
 	va_start(ap, msg);
304803
-	if (logging_to_syslog)
304803
-		vsyslog(LOG_ERR, msg, ap);
304803
-	else {
304803
-		vfprintf(stderr, msg, ap);
304803
+	if (logging_to_syslog) {
304803
+		if (prefixed_msg)
304803
+			vsyslog(LOG_ERR, prefixed_msg, ap);
304803
+		else
304803
+			vsyslog(LOG_INFO, msg, ap);
304803
+	} else {
304803
+		if (prefixed_msg)
304803
+			vfprintf(stderr, prefixed_msg, ap);
304803
+		else
304803
+			vfprintf(stderr, msg, ap);
304803
 		fputc('\n', stderr);
304803
 	}
304803
 	va_end(ap);
304803
+
304803
+	if (prefixed_msg)
304803
+		free(prefixed_msg);
304803
+
304803
 	return;
304803
 }
304803
 
304803
 void log_crit(unsigned logopt, const char *msg, ...)
304803
 {
304803
+	char *prefixed_msg;
304803
 	va_list ap;
304803
 
304803
+	prefixed_msg = prepare_attempt_prefix(msg);
304803
+
304803
 	va_start(ap, msg);
304803
-	if (logging_to_syslog)
304803
-		vsyslog(LOG_CRIT, msg, ap);
304803
-	else {
304803
-		vfprintf(stderr, msg, ap);
304803
+	if (logging_to_syslog) {
304803
+		if (prefixed_msg)
304803
+			vsyslog(LOG_CRIT, prefixed_msg, ap);
304803
+		else
304803
+			vsyslog(LOG_INFO, msg, ap);
304803
+	} else {
304803
+		if (prefixed_msg)
304803
+			vfprintf(stderr, prefixed_msg, ap);
304803
+		else
304803
+			vfprintf(stderr, msg, ap);
304803
 		fputc('\n', stderr);
304803
 	}
304803
 	va_end(ap);
304803
+
304803
+	if (prefixed_msg)
304803
+		free(prefixed_msg);
304803
+
304803
 	return;
304803
 }
304803
 
304803
 void log_debug(unsigned int logopt, const char *msg, ...)
304803
 {
304803
 	unsigned int opt_log = logopt & LOGOPT_DEBUG;
304803
+	char *prefixed_msg;
304803
 	va_list ap;
304803
 
304803
 	if (!do_debug && !opt_log)
304803
 		return;
304803
 
304803
+	prefixed_msg = prepare_attempt_prefix(msg);
304803
+
304803
 	va_start(ap, msg);
304803
-	if (logging_to_syslog)
304803
-		vsyslog(LOG_WARNING, msg, ap);
304803
-	else {
304803
-		vfprintf(stderr, msg, ap);
304803
+	if (logging_to_syslog) {
304803
+		if (prefixed_msg)
304803
+			vsyslog(LOG_WARNING, prefixed_msg, ap);
304803
+		else
304803
+			vsyslog(LOG_INFO, msg, ap);
304803
+	} else {
304803
+		if (prefixed_msg)
304803
+			vfprintf(stderr, prefixed_msg, ap);
304803
+		else
304803
+			vfprintf(stderr, msg, ap);
304803
 		fputc('\n', stderr);
304803
 	}
304803
 	va_end(ap);
304803
 
304803
+	if (prefixed_msg)
304803
+		free(prefixed_msg);
304803
+
304803
 	return;
304803
 }
304803
 
304803
 void logmsg(const char *msg, ...)
304803
 {
304803
+	char *prefixed_msg;
304803
 	va_list ap;
304803
+
304803
+	prefixed_msg = prepare_attempt_prefix(msg);
304803
+
304803
 	va_start(ap, msg);
304803
-	if (logging_to_syslog)
304803
-		vsyslog(LOG_CRIT, msg, ap);
304803
-	else {
304803
-		vfprintf(stderr, msg, ap);
304803
+	if (logging_to_syslog) {
304803
+		if (prefixed_msg)
304803
+			vsyslog(LOG_CRIT, prefixed_msg, ap);
304803
+		else
304803
+			vsyslog(LOG_INFO, msg, ap);
304803
+	} else {
304803
+		if (prefixed_msg)
304803
+			vfprintf(stderr, prefixed_msg, ap);
304803
+		else
304803
+			vfprintf(stderr, msg, ap);
304803
 		fputc('\n', stderr);
304803
 	}
304803
 	va_end(ap);
304803
+
304803
+	if (prefixed_msg)
304803
+		free(prefixed_msg);
304803
+
304803
 	return;
304803
 }
304803