OpenBSD src changes summary for 2017-07-27 ==========================================
gnu gnu/lib/libreadline gnu/usr.bin/clang lib/libc lib/librthread regress/lib regress/sbin regress/sys sbin/dhclient share/man sys/dev/pci sys/dev/wscons sys/kern sys/net sys/netinet sys/netinet6 sys/sys usr.bin/bgplg usr.bin/netstat usr.bin/tmux usr.bin/w usr.sbin/crunchgen usr.sbin/nsd usr.sbin/smtpd usr.sbin/snmpd == gnu =============================================================== 01/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu gnu ~ usr.bin/Makefile > ensure protoiize man pages installed in all cases (until we come > to a different decision later) > ok espie (deraadt@) ~ llvm/tools/clang/lib/Driver/Tools.cpp > push back -pie to the linker if -pie is specified to avoid an unused > argument > warning > ok kettenis@, deraadt@ (robert@) ~ llvm/tools/clang/include/clang/Sema/Sema.h ~ llvm/tools/clang/lib/Sema/SemaChecking.cpp ~ llvm/tools/clang/lib/Sema/SemaDeclAttr.cpp > teach clang about our syslog format attribute > ok deraadt@ (robert@) lib/libreadline ~ rldefs.h ~ rltty.c ~ savestring.c ~ terminal.c > fix warnings from missing headers. > okay deraadt@ (espie@) usr.bin/clang ~ lld/Makefile > don't double install ld, some people have already been lost trying > to get out of these makefiles! > okay kettenis@ (espie@) == lib =============================================================== 02/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libc ~ gdtoa/gethex.c ~ gdtoa/strtod.c ~ gdtoa/strtodg.c > Use stdrup, to avoid clang whining about the length parameters being > based upon input being used unsafely (they are safe) > ok millert kettenis (deraadt@) librthread ~ rthread.c ~ rthread_fork.c > bad things can (and will) happen if a threaded program calls fork() and > then strays off the path to exec(). one common manifestation of this > problem occurs in pthread_join(), so we can add a little check there. > first person to hit this in real life gets to change the error message. > (tedu@) == regress =========================================================== 03/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress lib ~ libc/setjmp-signal/setjmp-signal.c > Mark the invalid memory location as volatile, otherwise clang would > optimize the access to it with an illegal instruction. But the > tests needs a SIGSEGV, it would fail with SIGILL. (bluhm@) ~ libc/atexit/Makefile ~ libc/atexit/atexit_test.c ~ libc/cephes/ieetst.c ~ libc/cephes/mconf.h ~ libc/db/dbtest.c ~ libc/locale/Makefile ~ libc/locale/check_isw/Makefile ~ libc/locale/check_isw/check_isw.c ~ libc/locale/mbrtowc/Makefile ~ libc/locale/mbrtowc/test_mbrtowc.c ~ libc/locale/setlocale/Makefile ~ libc/locale/wcrtomb/Makefile ~ libc/locale/wcrtomb/test_wcrtomb.c ~ libc/malloc/malloc_ulimit1/malloc_ulimit1.c ~ libc/malloc/malloc_ulimit2/malloc_ulimit2.c ~ libc/sprintf/Makefile ~ libc/telldir/utils.c ~ libc/vis/vis_test.c > Fix all clang warnings in libc regress and cleanup some make files. > (bluhm@) ~ libc/getaddrinfo/Makefile ~ libc/getaddrinfo/answer > Sort expected getaddrinfo output in inet4 inet6 order, then the > test passes. (bluhm@) ~ libc/Makefile > Sort subdir, run getaddrinfo test, remove vax case. (bluhm@) sbin ~ route/Makefile > Work around for rttest8 that some times fail due to a reference count > race in the kernel. (mpi@) sys ~ kern/fcntl_dup/fcntl_dup.c > fcntl(F_DUPFD) needs an argument. Otherwise it fails with EINVAL > when compiled with clang. (bluhm@) ~ net/rtable/Makefile.inc ~ net/rtable/kern_compat.h ~ net/rtable/srp_compat.h ~ net/rtable/util.c ~ net/rtable/util.h > Compile and test MPATH code. (mpi@) == sbin ============================================================== 04/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin dhclient ~ kroute.c ~ privsep.h > Fold add_direct_route(), add_default_route() and > add_classless_static_routes() into set_routes(). One > less layer of abstraction and all five add_route() > invocations now in one place. > Clearly comment the route(8) command equivalents for each > add_route() invocation. (krw@) ~ kroute.c > Kill extra whitespace that snuck in. (krw@) == share ============================================================= 05/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man8/rc.d.8 > The -d flag prevents redirection of stdout (not stdin) and stderr > to /dev/null. > noticed by Matthew <chohag () jtan ! com> (tb@) == sys =============================================================== 06/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys dev/pci ~ drm/i915/i915_gem.c > Initialize pools with correct sizes. > This was safe because `struct drm_i915_gem_object' is bigger than the > other two. > ok kettenis@ (mpi@) dev/wscons ~ wstpad.c > Use variable edge masks (again). > The properties of edge areas will be more consistent even though > in some cases the driver remains in the dark about the exact > coordinate limits. (bru@) kern ~ uipc_socket.c ~ uipc_socket2.c > Assert that the KERNEL_LOCK() is held prior to call csignal() and > selwakeup(). > ok bluhm@ (mpi@) net ~ pipex_local.h > SEQ16_* macros weren't able to compare properly if one of the sequences is > wrapped around. Fix them to cast the proper width of integer when > comparison. found goda@ (yasuoka@) ~ pf_ioctl.c > For pf the anchor is a C string so ensure that the value passed in via > ioctl > is correctly NUL terminated. > Reported by Ilja Van Sprundel > With and OK bluhm@ (claudio@) ~ route.c > Export RTAX_IFP, RTAX_IFA and RTAX_LABEL to userland in rtdeletemsg(). > Reduce differences with rtm_miss(). > ok claudio@, bluhm@ (mpi@) netinet ~ ip_divert.c > Grab the KERNEL_LOCK() before calling sorwakeup(). > In the forwarding path, pf_test() is executed w/o KERNEL_LOCK() and > in case of divert end up calling sowakup(). However selwakup() and > csignal() are not yet ready to be executed w/o KERNEL_LOCK(). > ok bluhm@ (mpi@) netinet6 ~ ip6_divert.c > Grab the KERNEL_LOCK() before calling sorwakeup(). > In the forwarding path, pf_test() is executed w/o KERNEL_LOCK() and > in case of divert end up calling sowakup(). However selwakup() and > csignal() are not yet ready to be executed w/o KERNEL_LOCK(). > ok bluhm@ (mpi@) sys ~ systm.h > Stop doing an splsoftnet()/splx() dance inside the NET_LOCK(). > This will allow us to not carry a returned value when entering a critical > section. > ok bluhm@, visa@ (mpi@) == usr.bin =========================================================== 07/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin bgplg ~ misc.c > An array is never NULL; pointed out by clang. > OK deraadt (florian@) netstat ~ inet.c > uint32_t is never < 0; pointed out by clang. > While we originally get a short passed from the kernel, it's probably > better for debugging purposes to show what we are actually having > which is a uin32_t hence the %u format. > OK deraadt (florian@) tmux ~ format.c ~ tmux.1 > Add pane_at_left/right/top/bottom formats, from Amos Bird. (nicm@) w ~ w.c > kp can be NULL (eg, stale utmp entry), so adjust the comment. > Prompted by another diff from Klemens Nanni (jca@) == usr.sbin ========================================================== 08/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin crunchgen ~ crunched_main.c ~ crunchgen.c > Mute clang's zeal for whining by providing prototypes. (deraadt@) ~ crunchgen.c > insist on -Oz in the *.mk file > (note it's *after* the normal flags, so it will pre-empt a -O2) > okay deraadt@ (espie@) nsd ~ config.h.in > Add prototypes for prototypes for __b64_pton and __b64_ntop; pointed > out by clang. > This is a local diff we have to carry so config.h.in is good a place > as any and it's included everywhere. > OK sthen (florian@) smtpd ~ makemap.c ~ smtpctl.c ~ smtpd.h > smtpctl(8): Use an int to determine mode instead of __progname. > Ok millert@ gilles@ (sunil@) snmpd ~ snmpe.c > Heed the non-repeaters of GetBulkRequests and make sure to set > error-status and error-index to zero in the response PDU. > ok sthen@ (gerhard@) =============================================================================== _______________________________________________ odc mailing list [email protected] http://www.squish.net/mailman/listinfo/odc
