Source-Changes-HG archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

[src/trunk]: src/libexec/telnetd telnetd: Stop defining the same variables co...



details:   https://anonhg.NetBSD.org/src/rev/c73aa106ef18
branches:  trunk
changeset: 964737:c73aa106ef18
user:      kamil <kamil%NetBSD.org@localhost>
date:      Thu Aug 15 01:15:21 2019 +0000

description:
telnetd: Stop defining the same variables concurrently in bss and data

auth_level and require_secure_login were defined in two places:

 - global.c that picked it from headers and removed 'extern'
 - telnetd.c that initialized it to 0

line was defined twice: in global.c and sys_term.c with a non-zero value.
Remove the definition and initialization from sys_term.c and initialize
the variable early in main().

Detected during the build of telned with Address Sanitizer (MKSANITIZER).

Reviewed by <mrg>

diffstat:

 libexec/telnetd/sys_term.c |  11 ++---------
 libexec/telnetd/telnetd.c  |  16 ++++++----------
 2 files changed, 8 insertions(+), 19 deletions(-)

diffs (87 lines):

diff -r a9f31aa04c8a -r c73aa106ef18 libexec/telnetd/sys_term.c
--- a/libexec/telnetd/sys_term.c        Thu Aug 15 00:50:11 2019 +0000
+++ b/libexec/telnetd/sys_term.c        Thu Aug 15 01:15:21 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: sys_term.c,v 1.48 2018/12/13 04:29:18 maya Exp $       */
+/*     $NetBSD: sys_term.c,v 1.49 2019/08/15 01:15:21 kamil Exp $      */
 
 /*
  * Copyright (c) 1989, 1993
@@ -34,7 +34,7 @@
 #if 0
 static char sccsid[] = "@(#)sys_term.c 8.4+1 (Berkeley) 5/30/95";
 #else
-__RCSID("$NetBSD: sys_term.c,v 1.48 2018/12/13 04:29:18 maya Exp $");
+__RCSID("$NetBSD: sys_term.c,v 1.49 2019/08/15 01:15:21 kamil Exp $");
 #endif
 #endif /* not lint */
 
@@ -177,13 +177,6 @@
  *
  * Returns the file descriptor of the opened pty.
  */
-#ifndef        __GNUC__
-char *line = NULL16STR;
-#else
-static char Xline[] = NULL16STR;
-char *line = Xline;
-#endif
-
 
 static int ptyslavefd; /* for cleanopen() */
 
diff -r a9f31aa04c8a -r c73aa106ef18 libexec/telnetd/telnetd.c
--- a/libexec/telnetd/telnetd.c Thu Aug 15 00:50:11 2019 +0000
+++ b/libexec/telnetd/telnetd.c Thu Aug 15 01:15:21 2019 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: telnetd.c,v 1.55 2014/02/27 18:20:21 joerg Exp $       */
+/*     $NetBSD: telnetd.c,v 1.56 2019/08/15 01:15:21 kamil Exp $       */
 
 /*
  * Copyright (C) 1997 and 1998 WIDE Project.
@@ -65,7 +65,7 @@
 #if 0
 static char sccsid[] = "@(#)telnetd.c  8.4 (Berkeley) 5/30/95";
 #else
-__RCSID("$NetBSD: telnetd.c,v 1.55 2014/02/27 18:20:21 joerg Exp $");
+__RCSID("$NetBSD: telnetd.c,v 1.56 2019/08/15 01:15:21 kamil Exp $");
 #endif
 #endif /* not lint */
 
@@ -86,18 +86,10 @@
 #include <krb5/com_err.h>
 #endif
 
-#ifdef AUTHENTICATION
-int    auth_level = 0;
-#endif
-
 #if    defined(AUTHENTICATION) || defined(ENCRYPTION)
 #include <libtelnet/misc.h>
 #endif
 
-#ifdef SECURELOGIN
-int    require_secure_login = 0;
-#endif
-
 extern int require_hwpreauth;
 #ifdef KRB5
 extern krb5_context telnet_context;
@@ -164,6 +156,8 @@
 int
 main(int argc, char *argv[])
 {
+       static char Xline[] = NULL16STR;
+
        socklen_t fromlen;
        int on = 1;
        int ch;
@@ -171,6 +165,8 @@
        int tos = -1;
 #endif
 
+       line = Xline;
+
        pfrontp = pbackp = ptyobuf;
        netip = netibuf;
        nfrontp = nbackp = netobuf;



Home | Main Index | Thread Index | Old Index