OpenBSD src changes summary for 2016-10-10 ==========================================
bin/ed bin/mv distrib/sets lib/libutil regress/sys regress/usr.sbin sbin/fsck_msdos share/man sys/dev/pv sys/kern sys/msdosfs sys/net usr.bin/pkill usr.bin/ssh usr.bin/tmux usr.bin/uname usr.sbin/dhcpd usr.sbin/eigrpd usr.sbin/httpd usr.sbin/rcctl == bin =============================================================== 01/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/bin ed ~ sub.c > Remove infinite loop detection from the s-command. > If a zero-length match is found do the replacement and increment the start > point > for the next search by one. This allows for commands like s/^/- / > This brings the behaviour closer to the way sed and vi work. > OK schwarze@ (martijn@) mv ~ rm.c > remove some dead code that's only used in rm (tedu@) ~ rm.c > remove some more dead code. (previous diff from Jan Stary) (tedu@) ~ rm.c > zap extra includes (tedu@) ~ rm.c > don't need to worry about pre/post order and skipping directories. > remove more dead code (tedu@) == distrib =========================================================== 02/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib sets ~ lists/man/mi > sync (deraadt@) == lib =============================================================== 03/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libutil ~ imsg_init.3 > Fixup the example for msgbuf_write() and imsg_read() to check the > error cases for -1 and 0 explicitly (it initially only checked for -1, > I updated it to also check for 0, and rzalamena@ figured out that 0 > has to be checked in a differently). > OK millert@ rzalamena@ (reyk@) == regress =========================================================== 04/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress sys ~ net/etherip/Makefile ~ net/etherip/etherip_1.sh > Move check for sysctl settings from shell script to makefile to > handle skipping consistently like other tests. (bluhm@) usr.sbin ~ syslogd/args-client-tls-fake.pl ~ syslogd/args-tls-cafile-fake.pl > The TLS error message depends on the generated fake certificate. > Make the test's expectation less strict. (bluhm@) == sbin ============================================================== 05/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin fsck_msdos ~ boot.c > Rename BIOS parameter block field from bsPBP to bsBPB. This typo > has been fixed in FreeBSD in 2002. No binary change. > From Alexander von Gernler; OK krw@ (bluhm@) == share ============================================================= 06/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man9/bpf_mtap.9 > Fix typo "bfp". > OK dlg@ (bluhm@) == sys =============================================================== 07/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys dev/pv ~ hyperv.c ~ hypervreg.h ~ hypervvar.h > Reshuffle GUID device ID table and add $ tags to headers (mikeb@) kern ~ uipc_mbuf.c > copy the offset of data inside mbufs in m_copym(). > this is cheap since it is basic math. it also means that payloads > which have been aligned carefully will also be aligned in their > copy. > ok yasuoka@ claudio@ (dlg@) ~ uipc_mbuf.c > white space fixes. > no functional change (dlg@) msdosfs ~ bootsect.h ~ msdosfs_vfsops.c > Rename BIOS parameter block field from bsPBP to bsBPB. This typo > has been fixed in FreeBSD in 2002. No binary change. > From Alexander von Gernler; OK krw@ (bluhm@) net ~ if_ethersubr.c ~ if_vlan.c > ensure prepended ethernet headers are placed on ETHER_ALIGN boundaries, > even if m_prepend allocates a new mbuf in front of the current one. > this is done by asking M_PREPEND for ETHER_HDR_LEN + ETHER_ALIGN bytes, > and then calling m_adj(ETHER_ALIGN) after. > in the case M_PREPEND does not allocate a new mbuf and ends up with the > same layout as before. > in the allocation case, the requested length is provided on a long > boundary. an ETHER_HDR_LEN request would therefore be 6 bytes > allocated on a long boundary, when we want it to be at ETHER_ALIGN. > by asking for ETHER_HDR_LEN plus ETHER_ALIGN, we can m_adj ETHER_ALIGN > off to get us to the ETHER_ALIGN offset. > ok yasuoka@ mikeb@ (dlg@) == usr.bin =========================================================== 08/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin pkill ~ pkill.c > - mark delim variable as const. it is a literal string; > - no need to declare main() prototype; > - mark all functions as static; > - add __dead marker to usage(), since it doesn't return; > - zap extern *optarg/optind in main(), It is already done in unistd.h; > - return from main instead of exit(3) that enables stack protector; > - fix err() eval in pledge()s error path. > OK millert@ (gsoares@) ssh ~ kex.c > Unregister the KEXINIT handler after message has been received. > Otherwise an unauthenticated peer can repeat the KEXINIT and cause > allocation of up to 128MB -- until the connection is closed. > Reported by shilei-c at 360.cn (markus@) ~ kex.c TAGGED OPENBSD_6_0 > MFC: > Unregister the KEXINIT handler after message has been received. > Otherwise an unauthenticated peer can repeat the KEXINIT and cause > allocation of up to 128MB -- until the connection is closed. > Reported by shilei-c at 360.cn (markus@) ~ kex.c TAGGED OPENBSD_5_9 > MFC: > Unregister the KEXINIT handler after message has been received. > Otherwise an unauthenticated peer can repeat the KEXINIT and cause > allocation of up to 128MB -- until the connection is closed. > Reported by shilei-c at 360.cn (markus@) tmux ~ tmux.h ~ paste.c ~ arguments.c > Some more static. (nicm@) ~ cmd-resize-pane.c ~ layout.c ~ tmux.h > Do not allow the opposite pane to resize when dragging with the mouse > because it is not possible to keep the mouse on the border when the > minimum size is reached. (nicm@) ~ cfg.c ~ environ.c ~ format.c ~ grid.c ~ hooks.c ~ job.c ~ key-string.c ~ layout-set.c ~ mode-key.c ~ names.c ~ notify.c ~ options.c ~ procname.c ~ screen-redraw.c ~ screen.c ~ server-client.c ~ server-fn.c ~ server.c ~ session.c ~ signal.c ~ status.c ~ tmux.c ~ tty-keys.c ~ tty-term.c ~ tty.c > Loads more static, except for cmd-*.c and window-*.c. (nicm@) ~ cmd-attach-session.c ~ cmd-bind-key.c ~ cmd-break-pane.c ~ cmd-capture-pane.c ~ cmd-choose-buffer.c ~ cmd-choose-client.c ~ cmd-choose-tree.c ~ cmd-clear-history.c ~ cmd-command-prompt.c ~ cmd-confirm-before.c ~ cmd-copy-mode.c ~ cmd-detach-client.c ~ cmd-display-message.c ~ cmd-find-window.c ~ cmd-find.c ~ cmd-if-shell.c ~ cmd-join-pane.c ~ cmd-kill-pane.c ~ cmd-kill-server.c ~ cmd-kill-session.c ~ cmd-kill-window.c ~ cmd-list-buffers.c ~ cmd-list-clients.c ~ cmd-list-panes.c ~ cmd-list-sessions.c ~ cmd-list-windows.c ~ cmd-load-buffer.c ~ cmd-lock-server.c ~ cmd-move-window.c ~ cmd-new-session.c ~ cmd-new-window.c ~ cmd-paste-buffer.c ~ cmd-pipe-pane.c ~ cmd-refresh-client.c ~ cmd-rename-session.c ~ cmd-rename-window.c ~ cmd-resize-pane.c ~ cmd-respawn-pane.c ~ cmd-respawn-window.c ~ cmd-rotate-window.c ~ cmd-run-shell.c ~ cmd-save-buffer.c ~ cmd-select-layout.c ~ cmd-select-pane.c ~ cmd-select-window.c ~ cmd-send-keys.c ~ cmd-set-buffer.c ~ cmd-set-environment.c ~ cmd-set-hook.c ~ cmd-set-option.c ~ cmd-show-environment.c ~ cmd-show-messages.c ~ cmd-show-options.c ~ cmd-source-file.c ~ cmd-split-window.c ~ cmd-string.c ~ cmd-swap-pane.c ~ cmd-swap-window.c ~ cmd-switch-client.c ~ cmd-unbind-key.c ~ cmd-wait-for.c ~ window-clock.c > Add static in cmd-* and fix a few other nits. (nicm@) uname ~ uname.c > calls to uname(3) should be checked against non-negative value > upon successful and -1 on failure (as per POSIX). No functional change, > just > improves portability. > OK millert@ schwarze@ guenther@ (gsoares@) == usr.sbin ========================================================== 09/09 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin dhcpd ~ dhcp.c > Stop pretending we use RFC 3046/Option 82/Relay Agent Information. > RFC 3042 says servers that do not understand the option will not > echo it. > Plus, our desultory attempt at echoing was almost certainly broken > for OFFERs (use after free of packet data) and not even attempted > for NACKs. > ok millert@ (krw@) eigrpd ~ tlv.c > calls to uname(3) should be checked against non-negative value > upon successful and -1 on failure (as per POSIX). No functional change, > just > improves portability. > requested by guenther@ > looks correct to schwarze@ > OK renato@ (gsoares@) httpd ~ proc.c > Add more context to fatal*() messages so it makes easier to debug proc.c > internals. > ok phessler@ (rzalamena@) ~ proc.c > Fix msgbuf_write() usage idiom and modify the treatment for socket close > to exit gracefully instead of fatal()ing. > ok reyk@ (rzalamena@) ~ proc.c > Modify httpd(8)'s proc.c to use less file descriptors during the daemon > start up. To achieve this proc_init() initiates only the necessary pipes > between child and parent, allocate and distribute fds in proc_connect(). > In case of configuration checks ('-n') we do nothing in proc_init() and > proc_connect(). > ok reyk@ (rzalamena@) rcctl ~ rcctl.8 > improve .Bl -tag -width for -Tps mode; > patch from Jan Stary <hans at stare dot cz>; > "go ahead" jmc@ (schwarze@) =============================================================================== _______________________________________________ odc mailing list [email protected] http://www.squish.net/mailman/listinfo/odc
