--- embedaddon/sudo/common/sudo_debug.c	2013/07/22 10:46:11	1.1.1.3
+++ embedaddon/sudo/common/sudo_debug.c	2014/06/15 16:12:54	1.1.1.5
@@ -47,15 +47,16 @@
 #include <fcntl.h>
 #include <time.h>
 
+#define DEFAULT_TEXT_DOMAIN	"sudo"
+#include "gettext.h"		/* must be included before missing.h */
+
 #include "missing.h"
 #include "alloc.h"
-#include "error.h"
+#include "fatal.h"
 #include "sudo_plugin.h"
 #include "sudo_debug.h"
+#include "sudo_util.h"
 
-#define DEFAULT_TEXT_DOMAIN	"sudo"
-#include "gettext.h"
-
 /*
  * The debug priorities and subsystems are currently hard-coded.
  * In the future we might consider allowing plugins to register their
@@ -103,6 +104,7 @@ const char *const sudo_debug_subsystems[] = {
     "plugin",
     "hooks",
     "sssd",
+    "event",
     NULL
 };
 
@@ -118,6 +120,7 @@ static int sudo_debug_fd = -1;
 static int sudo_debug_mode;
 static char sudo_debug_pidstr[(((sizeof(int) * 8) + 2) / 3) + 3];
 static size_t sudo_debug_pidlen;
+static const int num_subsystems = NUM_SUBSYSTEMS;
 
 /*
  * Parse settings string from sudo.conf and open debugfile.
@@ -129,8 +132,12 @@ int sudo_debug_init(const char *debugfile, const char 
     char *buf, *cp, *subsys, *pri;
     int i, j;
 
+    /* Make sure we are not already initialized. */
+    if (sudo_debug_mode != SUDO_DEBUG_MODE_DISABLED)
+	return 1;
+
     /* Init per-subsystems settings to -1 since 0 is a valid priority. */
-    for (i = 0; i < NUM_SUBSYSTEMS; i++)
+    for (i = 0; i < num_subsystems; i++)
 	sudo_debug_settings[i] = -1;
 
     /* Open debug file if specified. */
@@ -155,8 +162,14 @@ int sudo_debug_init(const char *debugfile, const char 
 	sudo_debug_mode = SUDO_DEBUG_MODE_CONV;
     }
 
+    /* Stash the pid string so we only have to format it once. */
+    (void)snprintf(sudo_debug_pidstr, sizeof(sudo_debug_pidstr), "[%d] ",
+	(int)getpid());
+    sudo_debug_pidlen = strlen(sudo_debug_pidstr);
+
     /* Parse settings string. */
-    buf = estrdup(settings);
+    if ((buf = strdup(settings)) == NULL)
+	return 0;
     for ((cp = strtok(buf, ",")); cp != NULL; (cp = strtok(NULL, ","))) {
 	/* Should be in the form subsys@pri. */
 	subsys = cp;
@@ -181,12 +194,8 @@ int sudo_debug_init(const char *debugfile, const char 
 	    }
 	}
     }
-    efree(buf);
+    free(buf);
 
-    (void)snprintf(sudo_debug_pidstr, sizeof(sudo_debug_pidstr), "[%d] ",
-	(int)getpid());
-    sudo_debug_pidlen = strlen(sudo_debug_pidstr);
-
     return 1;
 }
 
@@ -212,28 +221,32 @@ sudo_debug_enter(const char *func, const char *file, i
 	"-> %s @ %s:%d", func, file, line);
 }
 
-void sudo_debug_exit(const char *func, const char *file, int line,
+void
+sudo_debug_exit(const char *func, const char *file, int line,
     int subsys)
 {
     sudo_debug_printf2(NULL, NULL, 0, subsys | SUDO_DEBUG_TRACE,
 	"<- %s @ %s:%d", func, file, line);
 }
 
-void sudo_debug_exit_int(const char *func, const char *file, int line,
+void
+sudo_debug_exit_int(const char *func, const char *file, int line,
     int subsys, int rval)
 {
     sudo_debug_printf2(NULL, NULL, 0, subsys | SUDO_DEBUG_TRACE,
 	"<- %s @ %s:%d := %d", func, file, line, rval);
 }
 
-void sudo_debug_exit_long(const char *func, const char *file, int line,
+void
+sudo_debug_exit_long(const char *func, const char *file, int line,
     int subsys, long rval)
 {
     sudo_debug_printf2(NULL, NULL, 0, subsys | SUDO_DEBUG_TRACE,
 	"<- %s @ %s:%d := %ld", func, file, line, rval);
 }
 
-void sudo_debug_exit_size_t(const char *func, const char *file, int line,
+void
+sudo_debug_exit_size_t(const char *func, const char *file, int line,
     int subsys, size_t rval)
 {
     /* XXX - should use %zu but our snprintf.c doesn't support it */
@@ -242,7 +255,8 @@ void sudo_debug_exit_size_t(const char *func, const ch
 }
 
 /* We use int, not bool, here for functions that return -1 on error. */
-void sudo_debug_exit_bool(const char *func, const char *file, int line,
+void
+sudo_debug_exit_bool(const char *func, const char *file, int line,
     int subsys, int rval)
 {
     if (rval == true || rval == false) {
@@ -254,14 +268,16 @@ void sudo_debug_exit_bool(const char *func, const char
     }
 }
 
-void sudo_debug_exit_str(const char *func, const char *file, int line,
+void
+sudo_debug_exit_str(const char *func, const char *file, int line,
     int subsys, const char *rval)
 {
     sudo_debug_printf2(NULL, NULL, 0, subsys | SUDO_DEBUG_TRACE,
 	"<- %s @ %s:%d := %s", func, file, line, rval ? rval : "(null)");
 }
 
-void sudo_debug_exit_str_masked(const char *func, const char *file, int line,
+void
+sudo_debug_exit_str_masked(const char *func, const char *file, int line,
     int subsys, const char *rval)
 {
     static const char stars[] = "********************************************************************************";
@@ -271,7 +287,8 @@ void sudo_debug_exit_str_masked(const char *func, cons
 	"<- %s @ %s:%d := %.*s", func, file, line, len, rval ? stars : "(null)");
 }
 
-void sudo_debug_exit_ptr(const char *func, const char *file, int line,
+void
+sudo_debug_exit_ptr(const char *func, const char *file, int line,
     int subsys, const void *rval)
 {
     sudo_debug_printf2(NULL, NULL, 0, subsys | SUDO_DEBUG_TRACE,
@@ -282,24 +299,34 @@ static void
 sudo_debug_write_conv(const char *func, const char *file, int lineno,
     const char *str, int len, int errno_val)
 {
-    /* Remove the newline at the end if appending extra info. */
-    if (str[len - 1] == '\n')
+    /* Remove trailing newlines. */
+    while (len > 0 && str[len - 1] == '\n')
 	len--;
 
-    if (func != NULL && file != NULL) {
-	if (errno_val) {
-	    sudo_printf(SUDO_CONV_DEBUG_MSG, "%.*s: %s @ %s() %s:%d",
-		len, str, strerror(errno_val), func, file, lineno);
+    if (len > 0) {
+	if (func != NULL && file != NULL) {
+	    if (errno_val) {
+		sudo_printf(SUDO_CONV_DEBUG_MSG, "%.*s: %s @ %s() %s:%d",
+		    len, str, strerror(errno_val), func, file, lineno);
+	    } else {
+		sudo_printf(SUDO_CONV_DEBUG_MSG, "%.*s @ %s() %s:%d",
+		    len, str, func, file, lineno);
+	    }
 	} else {
-	    sudo_printf(SUDO_CONV_DEBUG_MSG, "%.*s @ %s() %s:%d",
-		len, str, func, file, lineno);
+	    if (errno_val) {
+		sudo_printf(SUDO_CONV_DEBUG_MSG, "%.*s: %s",
+		    len, str, strerror(errno_val));
+	    } else {
+		sudo_printf(SUDO_CONV_DEBUG_MSG, "%.*s", len, str);
+	    }
 	}
-    } else {
-	if (errno_val) {
-	    sudo_printf(SUDO_CONV_DEBUG_MSG, "%.*s: %s",
-		len, str, strerror(errno_val));
+    } else if (errno_val) {
+	/* Only print error string. */
+	if (func != NULL && file != NULL) {
+	    sudo_printf(SUDO_CONV_DEBUG_MSG, "%s @ %s() %s:%d",
+		strerror(errno_val), func, file, lineno);
 	} else {
-	    sudo_printf(SUDO_CONV_DEBUG_MSG, "%.*s", len, str);
+	    sudo_printf(SUDO_CONV_DEBUG_MSG, "%s", strerror(errno_val));
 	}
     }
 }
@@ -311,8 +338,7 @@ sudo_debug_write_file(const char *func, const char *fi
     char *timestr, numbuf[(((sizeof(int) * 8) + 2) / 3) + 2];
     time_t now;
     struct iovec iov[12];
-    int iovcnt = 4;
-    bool need_newline = false;
+    int iovcnt = 3;
 
     /* Prepend program name and pid with a trailing space. */
     iov[1].iov_base = (char *)getprogname();
@@ -320,26 +346,25 @@ sudo_debug_write_file(const char *func, const char *fi
     iov[2].iov_base = sudo_debug_pidstr;
     iov[2].iov_len = sudo_debug_pidlen;
 
-    /* Add string along with newline if it doesn't have one. */
-    iov[3].iov_base = (char *)str;
-    iov[3].iov_len = len;
-    if (str[len - 1] != '\n')
-	need_newline = true;
+    /* Add string, trimming any trailing newlines. */
+    while (len > 0 && str[len - 1] == '\n')
+	len--;
+    if (len > 0) {
+	iov[iovcnt].iov_base = (char *)str;
+	iov[iovcnt].iov_len = len;
+	iovcnt++;
+    }
 
     /* Append error string if errno is specified. */
     if (errno_val) {
-	iov[iovcnt].iov_base = ": ";
-	iov[iovcnt].iov_len = 2;
-	iovcnt++;
+	if (len > 0) {
+	    iov[iovcnt].iov_base = ": ";
+	    iov[iovcnt].iov_len = 2;
+	    iovcnt++;
+	}
 	iov[iovcnt].iov_base = strerror(errno_val);
 	iov[iovcnt].iov_len = strlen(iov[iovcnt].iov_base);
 	iovcnt++;
-
-	/* Move newline to the end. */
-	if (!need_newline) {
-	    need_newline = true;
-	    iov[3].iov_len--;
-	}
     }
 
     /* If function, file and lineno are specified, append them. */
@@ -364,21 +389,12 @@ sudo_debug_write_file(const char *func, const char *fi
 	iov[iovcnt].iov_base = numbuf;
 	iov[iovcnt].iov_len = strlen(numbuf);
 	iovcnt++;
-
-	/* Move newline to the end. */
-	if (!need_newline) {
-	    need_newline = true;
-	    iov[3].iov_len--;
-	}
     }
 
-    /* Append newline as needed. */
-    if (need_newline) {
-	/* force newline */
-	iov[iovcnt].iov_base = "\n";
-	iov[iovcnt].iov_len = 1;
-	iovcnt++;
-    }
+    /* Append newline. */
+    iov[iovcnt].iov_base = "\n";
+    iov[iovcnt].iov_len = 1;
+    iovcnt++;
 
     /* Do timestamp last due to ctime's static buffer. */
     time(&now);
@@ -396,9 +412,6 @@ void
 sudo_debug_write2(const char *func, const char *file, int lineno,
     const char *str, int len, int errno_val)
 {
-    if (len <= 0)
-	return;
-
     switch (sudo_debug_mode) {
     case SUDO_DEBUG_MODE_CONV:
 	sudo_debug_write_conv(func, file, lineno, str, len, errno_val);
@@ -421,7 +434,7 @@ sudo_debug_vprintf2(const char *func, const char *file
     const char *fmt, va_list ap)
 {
     int buflen, pri, subsys, saved_errno = errno;
-    char *buf;
+    char *buf = NULL;
 
     if (!sudo_debug_mode)
 	return;
@@ -431,9 +444,8 @@ sudo_debug_vprintf2(const char *func, const char *file
     subsys = SUDO_DEBUG_SUBSYS(level);
 
     /* Make sure we want debug info at this level. */
-    if (subsys < NUM_SUBSYSTEMS && sudo_debug_settings[subsys] >= pri) {
-	buflen = vasprintf(&buf, fmt, ap);
-	va_end(ap);
+    if (subsys < num_subsystems && sudo_debug_settings[subsys] >= pri) {
+	buflen = fmt ? vasprintf(&buf, fmt, ap) : 0;
 	if (buflen != -1) {
 	    int errcode = ISSET(level, SUDO_DEBUG_ERRNO) ? saved_errno : 0;
 	    if (ISSET(level, SUDO_DEBUG_LINENO))
@@ -447,7 +459,19 @@ sudo_debug_vprintf2(const char *func, const char *file
     errno = saved_errno;
 }
 
+#ifdef NO_VARIADIC_MACROS
 void
+sudo_debug_printf_nvm(int pri, const char *fmt, ...)
+{
+    va_list ap;
+
+    va_start(ap, fmt);
+    sudo_debug_vprintf2(NULL, NULL, 0, pri, fmt, ap);
+    va_end(ap);
+}
+#endif /* NO_VARIADIC_MACROS */
+
+void
 sudo_debug_printf2(const char *func, const char *file, int lineno, int level,
     const char *fmt, ...)
 {
@@ -474,7 +498,7 @@ sudo_debug_execve2(int level, const char *path, char *
     subsys = SUDO_DEBUG_SUBSYS(level);
 
     /* Make sure we want debug info at this level. */
-    if (subsys >= NUM_SUBSYSTEMS || sudo_debug_settings[subsys] < pri)
+    if (subsys >= num_subsystems || sudo_debug_settings[subsys] < pri)
 	return;
 
     /* Log envp for debug level "debug". */
@@ -540,8 +564,16 @@ sudo_debug_execve2(int level, const char *path, char *
 }
 
 /*
- * Dup sudo_debug_fd to the specified value so we don't
- * close it when calling closefrom().
+ * Getter for the debug descriptor.
+ */
+int
+sudo_debug_fd_get(void)
+{
+    return sudo_debug_fd;
+}
+
+/*
+ * Setter for the debug descriptor.
  */
 int
 sudo_debug_fd_set(int fd)