CVS commit: src/libexec/telnetd

["Kamil Rytarowski" <kamil%netbsd.org@localhost>]

Module Name:    src
Committed By:   kamil
Date:           Thu Aug 15 01:15:21 UTC 2019

Modified Files:
        src/libexec/telnetd: sys_term.c telnetd.c

Log Message:
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>


To generate a diff of this commit:
cvs rdiff -u -r1.48 -r1.49 src/libexec/telnetd/sys_term.c
cvs rdiff -u -r1.55 -r1.56 src/libexec/telnetd/telnetd.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.