OpenBSD src changes summary for 2017-10-12 ==========================================
etc/rc lib/libc lib/libssl libexec/spamd regress/sys sbin/dhclient share/man sys/dev/acpi sys/dev/sdmmc sys/kern sys/net sys/sys usr.bin/mg usr.bin/tmux usr.sbin/spamdb usr.sbin/syspatch == etc =============================================================== 01/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/etc rc ~ rc > The testprogram for ld.so reordering is executed in tmpdir. > Move tmpdir for reordering library from /tmp to /usr/lib. > This allows to have /tmp mounted noexec. > prompted by reports on misc@ > OK deraadt@ tj@ tb@ (rpe@) == lib =============================================================== 02/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libc ~ string/bzero.3 > Add STANDARDS: denis@ spotted that it was missing. > OK deraadt@ jca@ jmc@ (schwarze@) libssl ~ ssl_locl.h > Drop prototypes for ssl23_*() functions, which no longer exist. (jsing@) ~ d1_meth.c ~ d1_srvr.c ~ ssl_locl.h ~ ssl_srvr.c > Fold dtls1_accept() into ssl_accept(), removing a lot of duplicated code. > With review/feedback from inoguchi@ (jsing@) ~ ssl_clnt.c ~ ssl_locl.h > Rename ssl3_client_hello() to ssl3_send_client_hello() for consistency. > (jsing@) == libexec =========================================================== 03/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/libexec spamd ~ spamd.c > Check for asprintf() returning -1 instead of assuming that the > pointer will remain unchanged when fails. OK deraadt@ (millert@) == regress =========================================================== 04/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress sys ~ kern/pledge/generic/Makefile ~ kern/pledge/generic/manager.c > Use a regex(3) to find the new pledge violation string in the dmesg. > (bluhm@) == sbin ============================================================== 05/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin dhclient ~ clparse.c ~ dhcpd.h ~ parse.c > Make parse_string() toe the new line by returning > 0 for failure, 1 for success, emitting a > single error message ("expecting string.") and > handling ';' better. > Don't leak memory when encountering pathological > config or lease files containing repeated instances > of an option or command with string data. (krw@) ~ clparse.c > Stop leaks of SSID strings. Whitespace fix. (krw@) ~ clparse.c > Don't leak string that overflows the option data buffer. (krw@) == share ============================================================= 06/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man4/pf.4 > some style fixes from ross l richardson; > checked by/ok bluhm (jmc@) ~ man9/mbuf.9 > Copy comment for M_LOOP from header file to mbuf(9). (bluhm@) == sys =============================================================== 07/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys dev/acpi ~ dsdt.c > Add missing size to free(); ok deraadt@ mpi@ (anton@) dev/sdmmc ~ sdmmc_io.c > Apparently one of the main concepts in the SDMMC I/O subsystem is that > the driver attached to an SDIO card always holds the lock and only > releases it once it detaches. Now with that in mind, some time ago > sdmmc_io_function_disable() and sdmmc_io_function_ready() were changed > to only assert the write lock and not take it, but not all of the tree > was converted. Change sdmmc_io_function_enable() as well, and remove > the enter/exit dance in the interrupt code. Apparently there is no > SDIO driver yet/anymore which would trigger those issues. > ok kettenis@ (patrick@) kern ~ uipc_mbuf.c > Move sysctl_mq() where it can safely mess with mbuf queue internals. > ok visa@, bluhm@, deraadt@ (mpi@) ~ kern_rwlock.c > Use a temporary variable in rw_status() to dereference only once the > volatile member of the struct. > Not forcing a memory read on every access, 3 in this function, might > reduce cache traffic in some cases. > Micro-optimization and diff provided by Mateusz Guzik. > ok visa@ (mpi@) ~ kern_pledge.c > Print the word pledge in the kernel log when there is a violation. > This should make it easier to figure out what is going on. Note > that the pledgecode it shows is only a guess which pledge(2) might > help. > OK deraadt@ semarie@ (bluhm@) net ~ if.c > Fix a singed vs unsigned comparison resulting in an overflow of the > routing socket. > When clang became the default compiler, `if_flags' from `struct ifnet' > was changed from "short" to "unsigned short", to silence a warning. > Sadly the copy of these flags on the stack was still a "short" which > made the flags comparison always true, which in turn made ifioctl() > generates a RTM_INFO message for many ioctl(2). > Since my last commit, the flag comparison is done for every ioctl(2). > This made the kernel generate at least one routing message per ioctl, > resulting in a lot of RTM_DESYNC. > RTM_DESYNC problem reported by krw@ thanks to dhclient(8)'s noisiness! > ok krw@, patrick@ (mpi@) ~ if.c ~ if_var.h > Move sysctl_mq() where it can safely mess with mbuf queue internals. > ok visa@, bluhm@, deraadt@ (mpi@) sys ~ exec_elf.h > Add Section Attirbute flag defines, some default values for n_type > and a couple of missing EM_* required to build devel/libdwarf and > devel/valgrind. > ok deraadt@, jasper@ (mpi@) ~ sysctl.h > Move sysctl_mq() where it can safely mess with mbuf queue internals. > ok visa@, bluhm@, deraadt@ (mpi@) == usr.bin =========================================================== 08/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin mg ~ cscope.c ~ grep.c > Replace fgetln(3) with getline(3) which is portable and less error prone. > From Scott Cheloha (scottcheloha AT gmail), thanks! > OK bcallah (florian@) tmux ~ format.c ~ server-fn.c ~ server.c ~ tmux.h ~ window.c > Show exit status and time in the remain-on-exit pane text, mostly from > Timo Boettcher in GitHub issue 1103. (nicm@) == usr.sbin ========================================================== 09/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin spamdb ~ spamdb.c > add -G to usage(); (jmc@) syspatch ~ syspatch.sh > Don't return 1 if the syspatch release directory does not exist; it just > means that no patch is available yet. (ajacoutot@) ~ syspatch.sh > Better error if installurl points to a local directory. (ajacoutot@) ~ syspatch.sh > Loosen up installurl(5) check: it's OK to have a trailing slash. > (ajacoutot@) ~ syspatch.sh > Simplify and explicitely state when we cannot access our mirror. > (ajacoutot@) =============================================================================== _______________________________________________ odc mailing list [email protected] http://www.squish.net/mailman/listinfo/odc
