OpenBSD src changes summary for 2017-02-12 ==========================================
distrib/miniroot distrib/sets distrib/special etc/Makefile etc/mtree/4.4BSD.dist etc/mtree/special gnu lib/libc regress/usr.bin sbin/dhclient share/man sys/arch/alpha/alpha sys/arch/amd64/amd64 sys/arch/arm/arm sys/arch/arm64/arm64 sys/arch/hppa/hppa sys/arch/i386/i386 sys/arch/m88k/m88k sys/arch/mips64/mips64 sys/arch/powerpc/powerpc sys/arch/sh/sh sys/arch/sparc64/sparc64 sys/dev sys/dev/ic sys/dev/pv sys/dev/usb sys/kern sys/sys sys/uvm usr.bin/mandoc == distrib =========================================================== 01/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib miniroot ~ install.sub > In autoinstall, no need to create /mnt/root/.ssh if the install script > provided an ssh public key, the directory is in the base set now. > From tb@, discussed with rpe@ and tb@ (landry@) sets ~ lists/base/mi ~ lists/etc/mi > sync (add /root/.ssh & /root/.ssh/authorized_keys) (landry@) ~ lists/comp/mi > sync (tb@) special ~ dhclient/Makefile > Switch from 'legacy' errwarn.c to standard daemon logging functions. > No objections heard. Feedback from millert@ guenther@ (krw@) == etc =============================================================== 02/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/etc Makefile ~ Makefile > As it's done in /etc/skel for new regular users, create an empty > /root/.ssh/authorized_keys file with correct permissions (0600 for the > file, 0700 for /root/.ssh dir). Since we encourage administrators to use > public keys only if they want to access root account via ssh, might > aswell make it easier, this will be particularly useful in > managed/provisioned environments (think ansible & others). > Note that administrators might get an e-mail from security(8) if the > file suddenly appears after an update - this is of course expected :) > ok tb@ sthen@ rpe@ ajacoutot@ (landry@) mtree/4.4BSD.dist ~ mtree/4.4BSD.dist > As it's done in /etc/skel for new regular users, create an empty > /root/.ssh/authorized_keys file with correct permissions (0600 for the > file, 0700 for /root/.ssh dir). Since we encourage administrators to use > public keys only if they want to access root account via ssh, might > aswell make it easier, this will be particularly useful in > managed/provisioned environments (think ansible & others). > Note that administrators might get an e-mail from security(8) if the > file suddenly appears after an update - this is of course expected :) > ok tb@ sthen@ rpe@ ajacoutot@ (landry@) mtree/special ~ mtree/special > Add /root/.ssh/authorized_keys to /etc/mtree/special so that security(8) > checks for the correct mode/ownership. > prodded by ajacoutot@ (landry@) == gnu =============================================================== 03/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu gnu ~ llvm/tools/clang/include/clang/Basic/DiagnosticSemaKinds.td > Disable -Wpointer-sign warnings per default > base gcc does the same. suggested by and ok jsg@ (stefan@) == lib =============================================================== 04/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libc ~ stdlib/malloc.3 > malloc does not *need* to return page-aligned objects for size >= > a page. This is not required by any standard and other malloc > implementation do not document (or implement) this. ok deraadt@ (otto@) == regress =========================================================== 05/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress usr.bin ~ mandoc/eqn/matrix/Makefile ~ mandoc/eqn/unary/sqrt.in ~ mandoc/eqn/unary/sqrt.out_ascii ~ mandoc/eqn/unary/sqrt.out_html + mandoc/eqn/matrix/empty.in + mandoc/eqn/matrix/empty.out_ascii + mandoc/eqn/matrix/empty.out_html > Do not access a NULL pointer if a matrix or square root are empty. > Crashes found by tb@ with afl(1). (schwarze@) == sbin ============================================================== 06/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin dhclient ~ Makefile ~ bpf.c ~ clparse.c ~ conflex.c ~ dhclient.c ~ dhcpd.h ~ dispatch.c ~ kroute.c ~ options.c ~ packet.c ~ parse.c ~ privsep.c + log.c + log.h > Switch from 'legacy' errwarn.c to standard daemon logging functions. > No objections heard. Feedback from millert@ guenther@ (krw@) - errwarn.c > Remove now unused file. (krw@) ~ bpf.c ~ clparse.c ~ dhclient.c ~ dispatch.c ~ kroute.c ~ options.c > Adjust lines that are too long. (krw@) ~ bpf.c ~ dhclient.c ~ dispatch.c ~ kroute.c ~ privsep.c > Eliminate most strerror() invocations by using log_warn() and fatal() > instead of log_warnx() and fatalx(). A few log_info() to log_warn() > for the same reason. > Suggested by millert@. (krw@) == share ============================================================= 07/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man9/Makefile ~ man9/fork1.9 + man9/thread_fork.9 > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) ~ man1/clang-local.1 > mention -Wpointer-sign being off by default (jsg@) ~ man1/Makefile > install clang-local(1) (jsg@) == sys =============================================================== 08/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/alpha/alpha ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/amd64/amd64 ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/arm/arm ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/arm64/arm64 ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/hppa/hppa ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/i386/i386 ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/m88k/m88k ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/mips64/mips64 ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/powerpc/powerpc ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/sh/sh ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) arch/sparc64/sparc64 ~ vm_machdep.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) dev ~ vscsi.c > Remove dead assignment and now unused variable. > Found by LLVM/Clang Static Analyzer. > ok dlg@ (chl@) dev/ic ~ rtwn.c > Fix an uninitialised return value in rtwn_ra_init(). Though nothing > currently tests the return value. > ok stsp@ (jsg@) dev/pv ~ xenstore.c > Remove incorrect if statement > Pointed out by jsg@ and Nathanael Rensen, <nathanael at > list ! polymorpheus ! com>, thanks! (mikeb@) dev/usb ~ if_smsc.c > The videocore portion of the raspberry pi which boots the arm cores and > runs the mailbox interface knows about a MAC address that appears to be > derived from a unique serial number along with the raspberry pi > foundation oui. > It modifies the device tree when booting to store the MAC address > in /axi/usb/hub/ethernet/mac-address so fetch and use this value > for the integrated smsc(4) Ethernet. > A different smsc adapter plugged into one of the USB ports > probes later with a different unit number and skips this path. (jsg@) ~ dwc2/dwc2.c ~ dwc2/dwc2_coreintr.c ~ dwc2/dwc2_hcdintr.c ~ dwc2/dwc2var.h > Replace dummy mutex asserts with real ones. > OK stsp@ (visa@) kern ~ init_main.c ~ kern_fork.c ~ kern_kthread.c ~ kern_sched.c > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) sys ~ proc.h > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) uvm ~ uvm_extern.h > Split up fork1(): > - FORK_THREAD handling is a totally separate function, thread_fork(), > that is only used by sys___tfork() and which loses the flags, func, > arg, and newprocp parameters and gains tcb parameter to guarantee > the new thread's TCB is set before the creating thread returns > - fork1() loses its stack and tidptr parameters > Common bits factor out: > - struct proc allocation and initialization moves to thread_new() > - maxthread handling moves to fork_check_maxthread() > - setting the new thread running moves to fork_thread_start() > The MD cpu_fork() function swaps its unused stacksize parameter for > a tcb parameter. > luna88k testing by aoyama@, alpha testing by dlg@ > ok mpi@ (guenther@) == usr.bin =========================================================== 09/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin mandoc ~ eqn_term.c > Do not access a NULL pointer if a matrix or square root are empty. > Crashes found by tb@ with afl(1). (schwarze@) =============================================================================== _______________________________________________ odc mailing list [email protected] http://www.squish.net/mailman/listinfo/odc
