OpenBSD src changes summary for 2015-06-07 to 2015-06-14 inclusive ==================================================================
distrib/armv7 distrib/notes distrib/octeon distrib/sets etc/etc.octeon/Makefile.inc etc/etc.octeon/disktab gnu/lib/libiberty gnu/usr.bin/binutils-2.17 lib/libc lib/libcrypto lib/libssl lib/libutil libexec/ld.so sbin/iked sbin/pfctl share/man share/snmp sys/arch/alpha/alpha sys/arch/amd64/amd64 sys/arch/amd64/include sys/arch/arm/arm sys/arch/arm/xscale sys/arch/armv7/armv7 sys/arch/armv7/conf sys/arch/armv7/exynos sys/arch/armv7/vexpress sys/arch/aviion/dev sys/arch/aviion/stand/boot sys/arch/i386/i386 sys/arch/i386/include sys/arch/landisk/stand/boot sys/arch/landisk/stand/xxboot sys/arch/macppc/macppc sys/arch/octeon/dev sys/arch/socppc/socppc sys/arch/sparc/stand/boot sys/arch/sparc64/sparc64 sys/dev/acpi sys/dev/gpio sys/dev/pci sys/dev/pckbc sys/dev/usb sys/kern sys/lib/libkern sys/lib/libsa sys/net sys/netinet sys/netinet6 sys/nfs sys/scsi sys/sys usr.bin/rcs usr.bin/tmux usr.sbin/dhcpd usr.sbin/httpd usr.sbin/ikectl usr.sbin/ldpd usr.sbin/npppd usr.sbin/pppd usr.sbin/relayctl usr.sbin/relayd usr.sbin/smtpd usr.sbin/snmpctl usr.sbin/snmpd usr.sbin/syslogd == distrib =========================================================== 01/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib armv7 ~ ramdisk/install.md > Move the scan_dmesg calls back into the functions. It can't be called > when the file is sourced as /var/run/dmesg.boot won't exist then. (jsg@) ~ ramdisk/install.md > expr isn't on the ramdisk use ksh for addition > problem spotted by jasper@ (jsg@) notes ~ octeon/hardware > add the edgerouter poe to the edgeroute lite paragraph (jasper@) ~ octeon/contents > add miniroot; note it's not further documented to prevent having to > reword/rewrite most of it again when usb support has landed. (jasper@) ~ alpha/xfer ~ amd64/xfer ~ armish/xfer ~ armv7/xfer ~ aviion/xfer ~ hppa/xfer ~ i386/xfer ~ landisk/xfer ~ loongson/xfer ~ luna88k/xfer ~ macppc/xfer ~ octeon/xfer ~ sgi/xfer ~ socppc/xfer ~ sparc/xfer ~ sparc64/xfer ~ vax/xfer ~ zaurus/xfer > FTP as an install method kicked the bucket some time ago. > ok miod@ (jasper@) octeon ~ Makefile + miniroot/Makefile > miniroot for octeon; tested on edgerouter lite with local usb storage > ok jmatthew@ miod@ (jasper@) sets ~ lists/base/md.socppc ~ lists/comp/md.socppc > sync (deraadt@) ~ lists/comp/mi > sync (deraadt@) ~ lists/man/mi > sync (deraadt@) == etc =============================================================== 02/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/etc etc.octeon/Makefile.inc ~ etc.octeon/Makefile.inc > add miniroot to MDEXT (jasper@) etc.octeon/disktab ~ etc.octeon/disktab > miniroot for octeon; tested on edgerouter lite with local usb storage > ok jmatthew@ miod@ (jasper@) == gnu =============================================================== 03/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu lib/libiberty ~ include/objalloc.h ~ src/objalloc.c > Fix CVE-2012-3509, an integer overflow in libiberty, leading to > heap-buffer overflow. > From Sebastian Trahm; OK deraadt@ (bluhm@) usr.bin/binutils-2.17 ~ gas/config/tc-arm.c ~ opcodes/arm-dis.c > Add more encodings of options for the armv7 barrier instructions and > allow non "sy"/0xf options for dmb. This omits the *ld options > available in armv8 running in a32 mode. Similiar changes made in > freebsd. > ok miod@ (jsg@) == lib =============================================================== 04/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libc ~ arch/arm/string/_memcpy.S ~ arch/arm/string/memset.S > At some point arm moved to a unified syntax for arm and thumb instructions. > While binutils supports both "unified" and "divided" syntax (defaulting > to divided) the integrated assembler in clang only supports unified names > so switch some files to unified syntax. Similiar changes were made in > bitrig and freebsd. No difference in objdump -d output. > tested on zaurus by deraadt@, ok miod@ (jsg@) ~ locale/setlocale.c > Remove unused parameter of load_locale_sub(). No assembly change on i386. > Diff from Sebastien Marie. objdump foo help from uwe (stsp@) ~ arch/amd64/SYS.h > Reuse SYSENTRY_HIDDEN() in SYSENTRY(); no functional changes. (uebayasi@) ~ gen/glob.c > in glob() initialize the glob_t before the first failure check. > from [email protected] > ok millert stsp (deraadt@) libcrypto ~ crypto/getentropy_linux.c > Remove unneeded sys/sysctl.h on linux. > This only provides the sysctl wrapper in glibc, which we do not use and is > not available in other libc implementations for Linux. Thanks to ncopa from > github. (bcook@) libssl ~ src/crypto/bn/bn_gf2m.c > Avoid an infinite loop that can be triggered by parsing an ASN.1 > ECParameters structure that has a specially malformed binary polynomial > field. > Issue reported by Joseph Barr-Pixton and fix based on OpenSSL. > Fixes CVE-2015-1788. > ok doug@ miod@ (jsing@) ~ src/crypto/x509/x509_vfy.c > Avoid a potential out-of-bounds read in X509_cmp_time(), due to missing > length checks. > Diff based on changes in OpenSSL. > Fixes CVE-2015-1789. > ok doug@ (jsing@) ~ src/crypto/cms/cms_smime.c > Avoid an infinite loop that can occur when verifying a message with an > unknown hash function OID. > Diff based on OpenSSL. > Fixes CVE-2015-1792 (however, this code is not enabled/built in LibreSSL). > ok doug@ miod@ (jsing@) ~ src/crypto/bn/bn_gf2m.c ~ src/crypto/cms/cms_smime.c ~ src/crypto/x509/x509_vfy.c TAGGED OPENBSD_5_7 > MFC: Fix several defects from OpenSSL. > These include: > CVE-2015-1788 - Malformed ECParameters causes infinite loop > CVE-2015-1789 - Exploitable out-of-bounds read in X509_cmp_time > CVE-2015-1792 - CMS verify infinite loop with unknown hash function > (jsing@) ~ src/crypto/bn/bn_gf2m.c ~ src/crypto/cms/cms_smime.c ~ src/crypto/x509/x509_vfy.c TAGGED OPENBSD_5_6 > MFC: Fix several defects from OpenSSL. > These include: > CVE-2015-1788 - Malformed ECParameters causes infinite loop > CVE-2015-1789 - Exploitable out-of-bounds read in X509_cmp_time > CVE-2015-1792 - CMS verify infinite loop with unknown hash function > (jsing@) ~ src/crypto/rsa/rsa_eay.c ~ src/crypto/x509/x509type.c ~ src/ssl/d1_clnt.c ~ src/ssl/d1_pkt.c ~ src/ssl/d1_srvr.c TAGGED OPENBSD_5_6 > Fix bad indenting in LibreSSL. > jsg@ noticed that some of the lines in libssl and libcrypto are not > indented properly. At a quick glance, it looks like it has a different > control flow than it really does. I checked the history in our tree and > in OpenSSL to make sure these were simple mistakes. > ok miod@ jsing@ (doug@) ~ src/ssl/bs_cbs.c TAGGED OPENBSD_5_6 > Reject long-form tags in CBS_peek_asn1_tag. > Currently, CBS only handles short-form tags. > ok miod@ jsing@ (doug@) ~ src/ssl/bytestring.h TAGGED OPENBSD_5_6 > Add comments about how the CBS constants are constructed. > Also, introduce a few more #defines to make it obvious. > ok miod@ jsing@ (doug@) ~ src/ssl/bs_cbb.c TAGGED OPENBSD_5_6 > When initial capacity is 0, always use NULL buffer. > malloc(0) is implementation defined and there's no reason to introduce > that ambiguity here. Added a few cosmetic changes in sizeof and free. > ok miod@ jsing@ (doug@) ~ src/ssl/bs_cbb.c TAGGED OPENBSD_5_6 > Explain the ASN.1 restriction that requires extra logic for encoding. > ok miod@ jsing@ (doug@) ~ src/ssl/bs_cbb.c TAGGED OPENBSD_5_6 > Split up the logic in CBB_flush to separately handle the lengths. > Also, add comments about assuming short-form. > ok miod@, tweak + ok jsing@ (doug@) libutil ~ imsg.c ~ imsg_init.3 > The correct semantic is to check msgbuf_write() for <= 0, not just < 0. > Fix one occurence in imsg_flush() and clarify it the man page. > Discussed with at least blambert@ jsg@ yasuoka@. > OK gilles@ (reyk@) == libexec =========================================================== 05/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/libexec ld.so ~ library_subr.c ~ ldconfig/library.c > Typos: equalivant, fucntion, libary. (miod@) ~ library_subr.c > breath->breadth (sthen@) == sbin ============================================================== 06/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin iked ~ chap_ms.h ~ dh.h ~ eap.h ~ iked.h ~ ikev2.h ~ types.h > Use "compliant" header guards by avoiding the reserved '_' namespace. > Pointed out by Markus Elfring > OK mikeb@ millert@ (reyk@) pfctl ~ pfctl.c > Allow rule ID filter to be specified for show states output > Tweak pfctl to respect the rule ID parameter (-R) specified > along with the show states (-s states) option to filter out > states that are not associated with a given rule from the > output. > ok sthen, benno (mikeb@) == share ============================================================= 07/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man4/gpiodcf.4 ~ man4/udcf.4 > HBG is no more so no need to mention it in the man pages. > OK jmc@ (claudio@) ~ man5/port-modules.5 > Update documentation for lang/go module > OK sthen@ (czarkoff@) ~ man9/Makefile ~ man9/hzto.9 ~ man9/timeout.9 ~ man9/tvtohz.9 > Remove hzto(9) manual pages and references; OK dlg (mikeb@) - man9/hzto.9 > Remove hzto(9) manual pages and references; OK dlg (mikeb@) ~ man7/mandoc_char.7 > Kill trailing blank; no text change. > Issue found by Tony Sim <y2s1982 at gmail dot com> > during today's BSDCan Doc Sprint. (schwarze@) ~ man4/pipex.4 > Add a missing .An macro. > Found by Enrico <Camarda at amnz dot org> > during the BSDCan Doc Sprint. (schwarze@) ~ man4/man4.armv7/Makefile + man4/man4.armv7/plrtc.4 + man4/man4.armv7/pluart.4 + man4/man4.armv7/sysreg.4 + man4/man4.armv7/vexpress.4 > add some initial vexpress man pages (jsg@) snmp ~ OPENBSD-PF-MIB.txt > Export new pf "no-route" error counter > OK sthen, blambert for the SNMP part (mikeb@) == sys =============================================================== 08/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/alpha/alpha ~ locore.s > In the copyout family of functions, if the address is out of range > ensure the register containing the proc pointer is initialized. > ok miod (deraadt@) arch/amd64/amd64 ~ cpu.c ~ machdep.c > Enable use of mwait in non-MP boxes and report # of C-substates up to C7, > truncating trailing zeros. > Testing by many as part of a larger change to use ACPI _CST objects > ok krw@ (guenther@) ~ mpbios.c > Add a default panic case to a switch statement where code after assumes > one of the cases was reached. Matches other parts of the mpbios code. > (jsg@) arch/amd64/include ~ cpu.h > Enable use of mwait in non-MP boxes and report # of C-substates up to C7, > truncating trailing zeros. > Testing by many as part of a larger change to use ACPI _CST objects > ok krw@ (guenther@) ~ specialreg.h > Add CR4_FSGSBASE (guenther@) ~ cpu.h > Parse _CST objects and use the C-states they describe when they're sane. > testing by many, particularly krw@ and jcs@ > tweaks by kettenis@ > ok deraadt@ (guenther@) arch/arm/arm ~ in_cksum_arm.S > At some point arm moved to a unified syntax for arm and thumb instructions. > While binutils supports both "unified" and "divided" syntax (defaulting > to divided) the integrated assembler in clang only supports unified names > so switch some files to unified syntax. Similiar changes were made in > bitrig and freebsd. No difference in objdump -d output. > tested on zaurus by deraadt@, ok miod@ (jsg@) arch/arm/xscale ~ i80321_clock.c > When investigating an uninitialised variable in the armv7 resettodr() > miod pointed out that time_second should be compared to 1 not 0 in the > md resettodr() functions as it is initialised to 1. > ok miod@ deraadt@ (jsg@) arch/armv7/armv7 ~ armv7var.h > Initial exynos4 bits. Among other things the gic isn't mapped correctly > on exynos4/5 yet as it isn't at the usual offset from periphbase. > ok bmercer@ (jsg@) ~ armv7var.h ~ platform.c > Add initial support for the ARM Versatile Express boards as emulated by > qemu with virtio memory ranges. > Unfortunately the vexpress-a9 and vexpress-a15 boards/targets have > different load addresses and memory maps. > Code for the PL011 UART and mmio virtio attachment from Patrick Wildt > in bitrig. (jsg@) ~ intr.c > When investigating an uninitialised variable in the armv7 resettodr() > miod pointed out that time_second should be compared to 1 not 0 in the > md resettodr() functions as it is initialised to 1. > ok miod@ deraadt@ (jsg@) arch/armv7/conf ~ Makefile.armv7 ~ files.armv7 > Add initial support for the ARM Versatile Express boards as emulated by > qemu with virtio memory ranges. > Unfortunately the vexpress-a9 and vexpress-a15 boards/targets have > different load addresses and memory maps. > Code for the PL011 UART and mmio virtio attachment from Patrick Wildt > in bitrig. (jsg@) ~ RAMDISK > enable agtimer on the ramdisk as well (jsg@) arch/armv7/exynos ~ exynos.c ~ exynos_machdep.c ~ files.exynos + exynos4.c > Initial exynos4 bits. Among other things the gic isn't mapped correctly > on exynos4/5 yet as it isn't at the usual offset from periphbase. > ok bmercer@ (jsg@) ~ exynos5.c > correct the uart irq numbers > ok bmercer@ (jsg@) arch/armv7/vexpress + files.vexpress + pl011.c + pl011reg.h + pl011var.h + sysreg.c + vexpress.c + vexpress_a15.c + vexpress_a9.c + vexpress_machdep.c + virtio_mmio.c > Add initial support for the ARM Versatile Express boards as emulated by > qemu with virtio memory ranges. > Unfortunately the vexpress-a9 and vexpress-a15 boards/targets have > different load addresses and memory maps. > Code for the PL011 UART and mmio virtio attachment from Patrick Wildt > in bitrig. (jsg@) ~ files.vexpress ~ vexpress.c ~ vexpress_a15.c ~ vexpress_a9.c + pl031.c > add a driver for the ARM PrimeCell PL031 RTC (jsg@) arch/aviion/dev ~ nvram.c > When investigating an uninitialised variable in the armv7 resettodr() > miod pointed out that time_second should be compared to 1 not 0 in the > md resettodr() functions as it is initialised to 1. > ok miod@ deraadt@ (jsg@) arch/aviion/stand/boot ~ Makefile > Bring a few routines from libkern in order to avoid linking against libgcc. > (miod@) arch/i386/i386 ~ cpu.c ~ machdep.c > Enable use of mwait in non-MP boxes and report # of C-substates up to C7, > truncating trailing zeros. > Testing by many as part of a larger change to use ACPI _CST objects > ok krw@ (guenther@) ~ mpbios.c > Add a default panic case to a switch statement where code after assumes > one of the cases was reached. Matches other parts of the mpbios code. > (jsg@) arch/i386/include ~ cpu.h > Enable use of mwait in non-MP boxes and report # of C-substates up to C7, > truncating trailing zeros. > Testing by many as part of a larger change to use ACPI _CST objects > ok krw@ (guenther@) ~ specialreg.h > Add CR4_FSGSBASE (guenther@) ~ cpu.h > Parse _CST objects and use the C-states they describe when they're sane. > testing by many, particularly krw@ and jcs@ > tweaks by kettenis@ > ok deraadt@ (guenther@) arch/landisk/stand/boot ~ Makefile > Build required bits from libkern rather than importing them from libgcc. > (miod@) arch/landisk/stand/xxboot ~ Makefile > Build required bits from libkern rather than importing them from libgcc. > (miod@) arch/macppc/macppc ~ clock.c > When investigating an uninitialised variable in the armv7 resettodr() > miod pointed out that time_second should be compared to 1 not 0 in the > md resettodr() functions as it is initialised to 1. > ok miod@ deraadt@ (jsg@) arch/octeon/dev ~ if_cnmacvar.h ~ if_cnmac.c > convert sc_sendq into an mbuf_list, and use ph_cookie rather than rcvif to > store the sgl address, allowing rcvif to be removed. > ok mpi@ dlg@ uebayasi@ (jmatthew@) arch/socppc/socppc ~ clock.c > When investigating an uninitialised variable in the armv7 resettodr() > miod pointed out that time_second should be compared to 1 not 0 in the > md resettodr() functions as it is initialised to 1. > ok miod@ deraadt@ (jsg@) arch/sparc/stand/boot ~ Makefile > Build __moddi3, __muldi3 and __qdivrem from libkern, and built no-pie, > instead > of getting them from libgcc.a, built pie. > This repairs boot blocks operation on sparc, as found the hard way by > sebastia@ (miod@) arch/sparc64/sparc64 ~ clock.c > When investigating an uninitialised variable in the armv7 resettodr() > miod pointed out that time_second should be compared to 1 not 0 in the > md resettodr() functions as it is initialised to 1. > ok miod@ deraadt@ (jsg@) dev/acpi ~ acpicpu.c ~ dsdt.h > Parse _CST objects and use the C-states they describe when they're sane. > testing by many, particularly krw@ and jcs@ > tweaks by kettenis@ > ok deraadt@ (guenther@) dev/gpio ~ files.gpio ~ gpiodcf.c > The Swiss Federal Government decided to shut down HBG at the end of 2011. > On 6 September 2012 at 12:02:00 UTC both antenna towers were demolished by > controlled explosives. So this is not coming back and we can tedu the > support for HBG form the DCF77 drivers. > Remided by mbalmer (claudio@) dev/pci ~ files.pci > mark the usb hci driver pci glue code as only needed by the pci attachment, > not all usb controllers. > this lets jsg build ehci on a platform that lacks a pci bus. > ok jsg@ (dlg@) ~ if_rtwn.c > Do not set "rcvif", if_input() does it for us. > Found the hardway by krw@ trying a diff that kills "rcvif". > ok stsp@ (mpi@) ~ autri.c > Use first 4 channels of the board. The previous channel setting > caused DMA on the wrong memory location during recording. > ok kettenis@ (ratchov@) ~ autri.c > Don't claim the autri(4) driver supports big-ending, signed 8-bit, > or unsigned 16-bit samples. Fixes sound on big endian machines. (ratchov@) ~ if_vio.c ~ viomb.c ~ virtio.c > remove uneeded pci includes (jsg@) ~ if_rtwn.c ~ if_rtwnreg.h > Fix rtwn(4) wifi LED support. Tested with rtwn(4) device in Thinkpad X220. > (stsp@) ~ if_iwm.c > Actually trigger iwm_init_task() from iwm_watchdog() as intended to give > recovery after device timeout a chance. Don't mess with the IFF_UP flag > in the watchdog since this isn't done anywhere except intel wifi drivers > which probably copied this pattern amongst each other. > ok kettenis@ (stsp@) ~ if_rtwn.c > Busy-wait a short while after sending a command to rtwn(4) firmware. > Fixes selection of initial TX rate. We're not stuck at 1Mbit/s anymore. > (stsp@) ~ if_rtwn.c ~ if_rtwnreg.h > Implement IQ calibration support for rtwn(4). Lots of black magic involved. > (stsp@) dev/pckbc ~ pms.c > pms(4): Don't match Elantech v4 devices with firmware versions 0xX7XXXX. > Apparently we don't support these touchpads properly so leave them in > PS/2 compat mode. Regression reported by Remi Locherer on bugs@. And stop > matching devices with firmware versions higher than 0xX8XXXX since we > cannot > be sure they work. > ok mpi@ (stsp@) dev/usb ~ udcf.c > The Swiss Federal Government decided to shut down HBG at the end of 2011. > On 6 September 2012 at 12:02:00 UTC both antenna towers were demolished by > controlled explosives. So this is not coming back and we can tedu the > support for HBG form the DCF77 drivers. > Remided by mbalmer (claudio@) ~ dwc2/dwc2.c > Ensure polled bulk, control and interrupt transfers actually poll, fixing > panics on shutdown with various usb sticks. > ok mpi@ (jmatthew@) ~ dwc2/dwc2.c > refuse to do polled isochronous transfers, as other usb controller drivers > do > requested by mpi@ (jmatthew@) ~ dwc2/dwc2_hcd.h ~ dwc2/dwc2_hcd.c ~ dwc2/dwc2_coreintr.c ~ dwc2/dwc2.h ~ dwc2/dwc2.c > rearrange delayed_work to avoid the use of container_of while leaving it in > more or less the same shape. > ok uebayasi@ (jmatthew@) ~ if_wi_usb.c > Typos: equalivant, fucntion, libary. (miod@) ~ if_athn_usb.c ~ if_axe.c ~ if_axen.c ~ if_axenreg.h ~ if_axereg.h ~ if_rsu.c ~ if_rum.c ~ if_rumreg.h ~ if_run.c ~ if_urtw.c ~ if_urtwn.c ~ if_wi_usb.c ~ if_wi_usb.h > Only match devices with a valid configuration. > Most of the WiFi/Ethernet USB adapter only have one configuration and > always > use its first interface. In order to improve USB descriptors parsing start > by reducing the number of places where a configuration is set. > Tests & ok stsp@ (mpi@) kern ~ kern_clock.c > Move hzto(9) to the attic; OK dlg (mikeb@) lib/libkern ~ arch/arm/memcpy.S ~ arch/arm/memset.S > At some point arm moved to a unified syntax for arm and thumb instructions. > While binutils supports both "unified" and "divided" syntax (defaulting > to divided) the integrated assembler in clang only supports unified names > so switch some files to unified syntax. Similiar changes were made in > bitrig and freebsd. No difference in objdump -d output. > tested on zaurus by deraadt@, ok miod@ (jsg@) lib/libsa ~ printf.c > Fix 1.26; kdoprnt() should not attempt to invoke va_end() at all, it's the > caller's responsibility to do so. (miod@) net ~ if.c ~ if.h ~ if_pflow.c ~ pf.c ~ pf_table.c > Introduce unhandled_af() for cases where code conditionally does > something based on an address family and later assumes one of the paths > was taken. This was initially just calls to panic until guenther > suggested a function to reduce the amount of strings needed. > This reduces the amount of noise with static analysers and acts > as a sanity check. > ok guenther@ bluhm@ (jsg@) ~ if_ethersubr.c > Move carp-related logic from ether_output() into carp_start(). > ok sthen@, phessler@ (mpi@) ~ if_bridge.c ~ if_ethersubr.c ~ if_vlan.c > Merge multiple copies of the code doing VLAN tag insertion back into > vlan_start(). > ok sthen@, phessler@ (mpi@) ~ if_trunk.c ~ trunklacp.c > Convert trunk(4) to if_input(). > ok dlg@ (mpi@) ~ if.c > Remove the hack to check if the received pointer has changed in an ifih > now that all drivers and pseudo-drivers are using if_input(). > if_input() is reentrant and is now the only place where we set `rcvif'. > (mpi@) ~ trunklacp.c > Add missing include. (mpi@) ~ if_bridge.c ~ pfkeyv2_convert.c > Move away from using hzto(9); OK dlg (mikeb@) ~ if_bridge.c > Remove superfluous splnet() protection. > ok lteo@ (mpi@) netinet ~ if_ether.c ~ ip_icmp.c ~ ip_input.c ~ ip_output.c ~ raw_ip.c ~ tcp_input.c ~ tcp_output.c ~ tcp_usrreq.c > Replace a bunch of == 0 with == NULL in pointer tests. Nuke some > annoying trailing, leading and embedded whitespace. No change to > .o files. > ok deraadt@ (krw@) ~ tcp_input.c ~ tcp_usrreq.c > Introduce unhandled_af() for cases where code conditionally does > something based on an address family and later assumes one of the paths > was taken. This was initially just calls to panic until guenther > suggested a function to reduce the amount of strings needed. > This reduces the amount of noise with static analysers and acts > as a sanity check. > ok guenther@ bluhm@ (jsg@) ~ ip_carp.c ~ ip_carp.h > Move carp-related logic from ether_output() into carp_start(). > ok sthen@, phessler@ (mpi@) ~ ipsec_input.c ~ ipsec_output.c > Move away from using hzto(9); OK dlg (mikeb@) netinet6 ~ icmp6.c ~ in6.c ~ in6.h ~ in6_cksum.c ~ in6_gif.c ~ in6_ifattach.c ~ in6_pcb.c ~ in6_src.c ~ ip6_divert.c ~ ip6_forward.c ~ ip6_input.c ~ ip6_mroute.c ~ ip6_mroute.h ~ ip6_output.c ~ nd6.c ~ nd6_rtr.c ~ raw_ip6.c ~ udp6_output.c > More damned eye searing whitespace. No change to .o files. (krw@) nfs ~ nfs_serv.c > Avoid double-free in error path by cribbing the HASBUF flag > logic from the rest of the kernel that deals with filename > lookups. > In snaps for some time. > Initially found by jsg@ > Prodded by deraadt@ (blambert@) ~ nfs_socket.c > Convert from hzto(9) to tvtohz(9); OK dlg (mikeb@) scsi ~ cd.c ~ mpath.c ~ mpath_hds.c ~ safte.c ~ scsi_all.h ~ scsi_base.c ~ scsi_changer.h ~ scsi_debug.h ~ scsi_disk.h ~ scsi_ioctl.c ~ scsi_tape.h ~ scsiconf.c ~ scsiconf.h ~ sd.c ~ st.c > More damned eye searing whitespace. (krw@) sys ~ systm.h > Move hzto(9) to the attic; OK dlg (mikeb@) == usr.bin =========================================================== 09/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin rcs ~ buf.c ~ ci.c ~ co.c ~ date.y ~ diff.c ~ diff3.c ~ rcs.c ~ rcsdiff.c ~ rcsnum.c ~ rcsparse.c ~ rcsprog.c ~ rcsutil.c ~ worklist.c ~ xmalloc.c ~ xmalloc.h > Convert xfree to free. From Fritjof Bornebusch. ok deraadt (nicm@) tmux ~ cmd-attach-session.c ~ cmd-new-session.c ~ cmd-switch-client.c ~ tmux.1 ~ tmux.h > Add -E flag when attaching or switching client to bypass > update-environment, from Steven Lu. (nicm@) ~ cmd-attach-session.c > Fix loop comparison broken in last commit, from Thomas Adam. (nicm@) ~ format.c > wp->tty is a char [] not a char * so it can't be NULL. From Thomas Adam. > (nicm@) ~ client.c ~ format.c ~ server-client.c ~ tmux.1 ~ tmux.h > Add a format for client PID (client_pid) and server PID (pid). Diff for > client_pid from Thomas Adam. (nicm@) == usr.sbin ========================================================== 10/10 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin dhcpd ~ dhcpd.conf.5 > Don't do manual whitespace adjustments inside semantic macros, > rather do it before the macros, and get rid of cargo cult escaping. > Both to make the code more robust and less ugly, no output change. > Ugly code reported by TJ at mrsk dot me. > OK jmc@ (schwarze@) httpd ~ server_fcgi.c > plug fd leak found by Todd Mortimer > ok claudio deraadt florian (jung@) ~ http.h > Use "compliant" header guards by avoiding the reserved '_' namespace. > Pointed out by Markus Elfring > OK mikeb@ millert@ (reyk@) ikectl ~ parser.h > Use "compliant" header guards by avoiding the reserved '_' namespace. > Pointed out by Markus Elfring > OK mikeb@ millert@ (reyk@) ldpd ~ lde_lib.c > Typos: equalivant, fucntion, libary. (miod@) npppd ~ npppd/npppd_radius.c > Typos: equalivant, fucntion, libary. (miod@) pppd ~ pppd.h ~ lcp.c > Perform a copy with a memmove for potentially overlapping regions. > Reported and fixed by Sergey Ryazanov <ryazanov ! s ! a at gmail ! com>, > thanks! (mikeb@) relayctl ~ parser.h > Use "compliant" header guards by avoiding the reserved '_' namespace. > Pointed out by Markus Elfring > OK mikeb@ millert@ (reyk@) relayd ~ relay.c ~ relay_http.c ~ relayd.h > Introduce a state on the ctl_relay_event struct. This makes it possible > to better track the connection state of a session and stops doing double > opens in certain situations using http relays. Using a state field to > simplify the logic since relay_connect() is called multiple times. > OK benno@, bluhm@ and running in production for more than a week (claudio@) ~ http.h ~ relayd.h > Use "compliant" header guards by avoiding the reserved '_' namespace. > Pointed out by Markus Elfring > OK mikeb@ millert@ (reyk@) ~ relay.c ~ relay_http.c ~ relayd.h > To match relayd's style, use an explicit enum with prefixed names for > the states that Claudio introduced. No functional change. > OK claudio@ benno@ (reyk@) smtpd ~ control.c ~ mproc.c TAGGED OPENBSD_5_6 > errata 25 (gilles@) ~ control.c ~ mproc.c TAGGED OPENBSD_5_6 > local user can cause smtpd to fail by sending invalid imsg to control sock > (gilles@) ~ control.c ~ mproc.c TAGGED OPENBSD_5_7 > errata 8 (gilles@) snmpctl ~ parser.h > Use "compliant" header guards by avoiding the reserved '_' namespace. > Pointed out by Markus Elfring > OK mikeb@ millert@ (reyk@) snmpd ~ mib.c ~ mib.h > Export new pf "no-route" error counter > OK sthen, blambert for the SNMP part (mikeb@) ~ mib.h ~ snmp.h ~ snmpd.h > Use "compliant" header guards by avoiding the reserved '_' namespace. > Pointed out by Markus Elfring > OK mikeb@ millert@ (reyk@) syslogd ~ syslogd.c > If fork fails, print an error message before exit. > OK doug@ djm@ (bluhm@) ~ syslogd.c > Close the lock pipe before dupping /dev/null to stdio. > This works even if the file descriptor is 0 or 1 or 2. > input and OK millert@ (bluhm@) =============================================================================== _______________________________________________ owc mailing list [email protected] http://www.squish.net/mailman/listinfo/owc
