- (djm) OpenBSD CVS Sync

   - djm@cvs.openbsd.org 2003/05/24 09:02:22
     [log.c]
     pass logged data through strnvis; ok markus

diff --git a/ChangeLog b/ChangeLog
index 242f21d9f3454afa2dc5283542d014f8c654e30f..e12d1032e48d00a501ef3ccfae9d6c534a36e98d 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+20030525
+- (djm) OpenBSD CVS Sync
+   - djm@cvs.openbsd.org 2003/05/24 09:02:22
+     [log.c]
+     pass logged data through strnvis; ok markus
+
 20030524
  - (dtucker) Correct --osfsia in INSTALL.  Patch by skeleten at shillest.net
 

diff --git a/log.c b/log.c
index 5a28438f97859f8db09c17c6928d2211263e4d24..fb89a965bff1a493a223119cbd4d7b89c543c728 100644 (file)
--- a/log.c
+++ b/log.c
@@ -34,12 +34,13 @@
  */
 
 #include "includes.h"
-RCSID("$OpenBSD: log.c,v 1.27 2003/05/18 23:22:01 deraadt Exp $");
+RCSID("$OpenBSD: log.c,v 1.28 2003/05/24 09:02:22 djm Exp $");
 
 #include "log.h"
 #include "xmalloc.h"
 
 #include <syslog.h>
+#include <vis.h>
 
 static LogLevel log_level = SYSLOG_LEVEL_INFO;
 static int log_on_stderr = 1;
@@ -390,16 +391,14 @@ do_log(LogLevel level, const char *fmt, va_list args)
        } else {
                vsnprintf(msgbuf, sizeof(msgbuf), fmt, args);
        }
-       /* Escape magic chars in output. */
        strnvis(fmtbuf, msgbuf, sizeof(fmtbuf), VIS_SAFE|VIS_OCTAL);
-       
        if (log_on_stderr) {
-               snprintf(fmtbuf, sizeof fmtbuf, "%s\r\n", msgbuf);
-               write(STDERR_FILENO, fmtbuf, strlen(fmtbuf));
+               snprintf(msgbuf, sizeof msgbuf, "%s\r\n", fmtbuf);
+               write(STDERR_FILENO, msgbuf, strlen(msgbuf));
        } else {
 #ifdef OPENLOG_R
                openlog_r(argv0 ? argv0 : __progname, LOG_PID, log_facility, &sdata);
-               syslog_r(pri, &sdata, "%.500s", msgbuf);
+               syslog_r(pri, &sdata, "%.500s", fmtbuf);
                closelog_r(&sdata);
 #else
                openlog(argv0 ? argv0 : __progname, LOG_PID, log_facility);