OpenBSD src changes summary for 2017-04-27 ==========================================
gnu lib/libc lib/libkeynote lib/libpcap lib/libtls libexec/ftpd libexec/login_radius libexec/rpc.rusersd regress/usr.bin share/man share/mk sys/arch/alpha/alpha sys/arch/amd64/amd64 sys/arch/amd64/include sys/arch/arm/arm sys/arch/arm/armv7 sys/arch/arm/conf sys/arch/arm/include sys/arch/arm/mainbus sys/arch/arm/simplebus sys/arch/arm64/arm64 sys/arch/arm64/conf sys/arch/arm64/dev sys/arch/arm64/include sys/dev/acpi sys/dev/pci sys/kern usr.bin/ssh usr.sbin/rebound == gnu =============================================================== 01/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu gnu ~ gcc/gcc/dwarf2.h ~ gcc/gcc/dwarf2out.c ~ gcc/gcc/config/i386/i386.c ~ gcc/gcc/config/i386/i386.opt ~ gcc/gcc/doc/invoke.texi > add support for -msave-args in gcc on amd64 > i got sick of not having arguments in ddb stack traces on amd64, > which is because amd64 passes arguments in registers, and it's > impossible to figure out where they go without dwarf info, and when > you have dwarf info it is complicated. > solaris has a simple solution for this. they tweaked their compilers > to accept an -msave-args option which makes functions store their > arguments on the stack, while maintaining compatability with the > System V AMD64 ABI. tools (eg, ddb) can then look at the stack to > get access to function arguments in traces. > this ports their changes to gcc 3 to our gcc. > ok deraadt@ (dlg@) == lib =============================================================== 02/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libc ~ net/ip6opt.c > Remove "len < 0" check; len is socklen_t (uint32_t) so can't be > negative. Quiets a warning from clang. OK bluhm@ (millert@) ~ nls/catopen.c > ntohl() returns uint32_t so it cannot be < 0. Since we're storing > the result in an int check for > INT_MAX instead. OK bluhm@ (millert@) libkeynote ~ keynote.y > Remove useless check for ks_env_table being NULL. Since ks_env_table > is an array embedded in struct keynote_session it can never be NULL. > Quiets a clang warning. (millert@) libpcap ~ gencode.h > Remove the "volatile" attribute from bpf_error(). The "volatile" > attribute is an obsolete way of saying "noreturn". clang doesn't > recognize it and the function is already marked __dead so it is > superfluous anyway. (millert@) libtls ~ man/tls_client.3 ~ man/tls_init.3 > tls_free(3) and tls_config_free(3) accept NULL; > patch from Matthew Martin <phy1729 at gmail dot com>, > slightly tweaked by me (schwarze@) == libexec =========================================================== 03/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/libexec ftpd ~ ftpcmd.y > Don't send multiple error messages in response to a single command > While a few commands (such as HELP and NOOP) are allowed in between > USER and PASS, most of the others are not. However if such command > is issued, ftpd should abort the command evaluation early and reply > with "530 Please login with USER and PASS." > From form@, with input from deraadt@ and OK millert@ (mikeb@) login_radius ~ login_radius.c > challenge is an array, not a pointer, therefore cannot be NULL. > Quiets a clang warning. (millert@) rpc.rusersd ~ rusers_proc.c > Change len arg of getidle() from size_t to int since it is used as > a length parameter to a "%.*s" printf format. Quiets a compiler > warning (both gcc and clang). (millert@) == regress =========================================================== 04/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress usr.bin ~ mdoclint/mdoclint > Detect links to self by parsing .Nm in SYNOPSIS. > Functionality suggested and patch OK'ed by jmc@. > Patch also accepted upstream in pkgsrc by wiz@. (schwarze@) ~ mdoclint/mdoclint > Merge trivial patches from pkgsrc to get rid of gratuitious differences; > no functional change on OpenBSD: rev.s 1.50, 1.52, 1.62, 1.63 > Of course, keep intentional differences. (schwarze@) == share ============================================================= 05/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man1/gcc-local.1 > add support for -msave-args in gcc on amd64 > i got sick of not having arguments in ddb stack traces on amd64, > which is because amd64 passes arguments in registers, and it's > impossible to figure out where they go without dwarf info, and when > you have dwarf info it is complicated. > solaris has a simple solution for this. they tweaked their compilers > to accept an -msave-args option which makes functions store their > arguments on the stack, while maintaining compatability with the > System V AMD64 ABI. tools (eg, ddb) can then look at the stack to > get access to function arguments in traces. > this ports their changes to gcc 3 to our gcc. > ok deraadt@ (dlg@) ~ man4/iwm.4 > iwm(4) monitor mode support was backed out for now. (stsp@) ~ man1/gcc-local.1 > spelling; (jmc@) mk ~ bsd.lib.mk > syspatch needs to create shared libs with the same object sequence instead > of randomizing the order so let's read the libs on the system and link the > same way in case a syspatch is being built (robert@) == sys =============================================================== 06/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/alpha/alpha ~ machdep.c > Unifdef KADB. > ok deraadt@ (mpi@) arch/amd64/amd64 ~ fpu.c ~ vmm.c ~ vmm_support.S > vmm(4): proper save/restore of FPU context during entry/exit. > tested by reyk, dcoppa, and a few others. > ok kettenis@ on the fpu bits > ok deraadt@ on the vmm bits (mlarkin@) ~ vmm.c > use a more descriptive value from the VEI_DIR_xxx enum instead of a > hardcoded number. no functional change (mlarkin@) arch/amd64/include ~ cpufunc.h ~ fpu.h ~ vmmvar.h > vmm(4): proper save/restore of FPU context during entry/exit. > tested by reyk, dcoppa, and a few others. > ok kettenis@ on the fpu bits > ok deraadt@ on the vmm bits (mlarkin@) ~ vmmvar.h > rename a struct that was denoted as "VMX only" to make it more clear > that it can be used in SVM and VMX. > no functional change (mlarkin@) arch/arm/arm ~ cpu.c > Bring over the changes to mainbus(4) and simplebus(4) from arm64. > (kettenis@) arch/arm/armv7 ~ armv7_a4x_io.S > Use (32-bit) word-sized access in the a4x bus space routine even if only > a byte or a half-word is needed. Certain implementations of the Synopsis > Designware copy-and-paste logic blocks don't respond to transactions that > are smaller than a word. > Fixes the serial console on boards with a Rockchip RK3288. (kettenis@) arch/arm/conf ~ files.arm > Bring over the changes to mainbus(4) and simplebus(4) from arm64. > (kettenis@) arch/arm/include ~ armreg.h ~ cpu.h > Bring over the changes to mainbus(4) and simplebus(4) from arm64. > (kettenis@) arch/arm/mainbus - cpu_mainbus.c ~ mainbus.c > Bring over the changes to mainbus(4) and simplebus(4) from arm64. > (kettenis@) arch/arm/simplebus ~ simplebus.c > Bring over the changes to mainbus(4) and simplebus(4) from arm64. > (kettenis@) arch/arm64/arm64 + cpu.c > Add code to identify the CPUs on arm64 systems. The primary CPU is > attached > and identified early on. For the secondary CPUs this happens late, such > that > the drivers we need to spin up CPUs, such as psci(4), will be available. > This also fixes some code in simplebus(4) where the return value of > OF_getprop() was not properly checked. > Heavily based on an earlier diff from drahn@. > ok drahn@, jsg@ (kettenis@) arch/arm64/conf ~ GENERIC ~ RAMDISK ~ files.arm64 > Add code to identify the CPUs on arm64 systems. The primary CPU is > attached > and identified early on. For the secondary CPUs this happens late, such > that > the drivers we need to spin up CPUs, such as psci(4), will be available. > This also fixes some code in simplebus(4) where the return value of > OF_getprop() was not properly checked. > Heavily based on an earlier diff from drahn@. > ok drahn@, jsg@ (kettenis@) arch/arm64/dev ~ mainbus.c ~ mainbus.h ~ simplebus.c > Add code to identify the CPUs on arm64 systems. The primary CPU is > attached > and identified early on. For the secondary CPUs this happens late, such > that > the drivers we need to spin up CPUs, such as psci(4), will be available. > This also fixes some code in simplebus(4) where the return value of > OF_getprop() was not properly checked. > Heavily based on an earlier diff from drahn@. > ok drahn@, jsg@ (kettenis@) ~ arm64_bus_space.c > Use (32-bit) word-sized access in the a4x bus space routine even if only > a byte or a half-word is needed. Certain implementations of the Synopsis > Designware copy-and-paste logic blocks don't respond to transactions that > are smaller than a word. > Fixes the serial console on boards with a Rockchip RK3288. (kettenis@) arch/arm64/include ~ armreg.h > Add code to identify the CPUs on arm64 systems. The primary CPU is > attached > and identified early on. For the secondary CPUs this happens late, such > that > the drivers we need to spin up CPUs, such as psci(4), will be available. > This also fixes some code in simplebus(4) where the return value of > OF_getprop() was not properly checked. > Heavily based on an earlier diff from drahn@. > ok drahn@, jsg@ (kettenis@) dev/acpi ~ acpicpu.c > track size of pss allocation, so that length can be passed to free(9) > ok mlarkin (deraadt@) dev/pci ~ if_iwm.c > Revert if_iwm.c to the state of r1.170 because ETOOMANYDRAGONSATONCE. > Performance regression reported by deraadt@ and uvm_fault reported by > fcambus@. > I will be poking further at this out of tree. (stsp@) ~ if_iwm.c > Restore if_iwm.c r1.173 (mira retry change) which got incarcerated by > the backout police even though it wasn't responsible for anything. > pointed out by tb@ (stsp@) kern ~ subr_prf.c > Unifdef KADB. > ok deraadt@ (mpi@) ~ kern_sysctl.c > Enforce that sysctl kern.somaxconn and sominconn can only be set > to valid values. The so_qlimit is type short. > report Dillon Jay Pena; OK deraadt@ (bluhm@) == usr.bin =========================================================== 07/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin ssh ~ scp.c > Avoid potential signed int overflow when parsing the file size. > Use strtoul() instead of parsing manually. OK djm@ (millert@) ~ servconf.c > remove a static array unused since rev 1.306 spotted by clang > ok djm@ (jsg@) == usr.sbin ========================================================== 08/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin rebound ~ rebound.c > clang warns about some of the strlcpy arguments here, which aren't the > typical idiom because there's invisible size dependencies. rewrite some > of it to use memcpy, which makes clear the lengths are the same. (tedu@) =============================================================================== _______________________________________________ odc mailing list [email protected] http://www.squish.net/mailman/listinfo/odc
