log: use lxc_unix_epoch_to_utc()

This allows us to generate nice timestamps in a thread-safe manner without relying on locale touching functions from any libc. Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

log: use lxc_unix_epoch_to_utc()
1d3fef81 · Christian Brauner · Stéphane Graber · 2bfafb07 · 1d3fef81
Commit 1d3fef81 authored Nov 28, 2016 by Christian Brauner Committed by Stéphane Graber Nov 30, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 10 deletions

log.c src/lxc/log.c +5 -10

No files found.
--- a/src/lxc/log.c
+++ b/src/lxc/log.c
@@ -136,10 +136,9 @@ static int log_append_logfile(const struct lxc_log_appender *appender,
 			      struct lxc_log_event *event)
 {
 	char buffer[LXC_LOG_BUFFER_SIZE];
-	struct timespec zero_timespec = {0};
+	char date_time[LXC_LOG_TIME_SIZE];
-	int ret, n;
+	int n;
 	int fd_to_use = -1;
-	char nanosec[__LXC_NUMSTRLEN];
 #ifndef NO_LXC_CONF
 	if (!lxc_log_use_global_fd && current_config)
@@ -152,19 +151,15 @@ static int log_append_logfile(const struct lxc_log_appender *appender,
 	if (fd_to_use == -1)
 		return 0;
-	ret = snprintf(nanosec, __LXC_NUMSTRLEN, "%ld", event->timestamp.tv_nsec);
+	if (lxc_unix_epoch_to_utc(date_time, LXC_LOG_TIME_SIZE, &event->timestamp) < 0)
-	if (ret < 0 || ret >= LXC_LOG_TIME_SIZE)
 		return 0;
 	n = snprintf(buffer, sizeof(buffer),
-			"%15s%s%s [ %.0f.%.6s ] %-8s %s - %s:%s:%d - ",
+			"%15s%s%s %s %-8s %s - %s:%s:%d - ",
 			log_prefix,
 			log_vmname ? " " : "",
 			log_vmname ? log_vmname : "",
-			/* sec_since_epoch: Safely convert time_t type to
+			date_time,
-			 * double. */
-			difftime(event->timestamp.tv_sec, zero_timespec.tv_sec),
-			nanosec,
 			lxc_log_priority_to_string(event->priority),
 			event->category,
 			event->locinfo->file, event->locinfo->func,