no need to call log_init to reinitialise logged PID in child sessions,
authordjm <djm@openbsd.org>
Sat, 24 Jun 2017 05:24:11 +0000 (05:24 +0000)
committerdjm <djm@openbsd.org>
Sat, 24 Jun 2017 05:24:11 +0000 (05:24 +0000)
since we haven't called openlog() in log_init() since 1999; ok markus@

usr.bin/ssh/session.c

index b924901..32046ef 100644 (file)
@@ -1,4 +1,4 @@
-/* $OpenBSD: session.c,v 1.288 2017/05/31 09:15:42 deraadt Exp $ */
+/* $OpenBSD: session.c,v 1.289 2017/06/24 05:24:11 djm Exp $ */
 /*
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
  *                    All rights reserved
@@ -118,7 +118,6 @@ static int session_pty_req(Session *);
 /* import */
 extern ServerOptions options;
 extern char *__progname;
-extern int log_stderr;
 extern int debug_flag;
 extern u_int utmp_len;
 extern int startup_pipe;
@@ -345,10 +344,6 @@ do_exec_no_pty(Session *s, const char *command)
        case 0:
                is_child = 1;
 
-               /* Child.  Reinitialize the log since the pid has changed. */
-               log_init(__progname, options.log_level,
-                   options.log_facility, log_stderr);
-
                /*
                 * Create a new session and process group since the 4.4BSD
                 * setlogin() affects the entire process group.
@@ -484,9 +479,6 @@ do_exec_pty(Session *s, const char *command)
                close(fdout);
                close(ptymaster);
 
-               /* Child.  Reinitialize the log because the pid has changed. */
-               log_init(__progname, options.log_level,
-                   options.log_facility, log_stderr);
                /* Close the master side of the pseudo tty. */
                close(ptyfd);