OpenBSD src changes summary for 2015-06-21 to 2015-06-28 inclusive ==================================================================
bin/chmod bin/ed bin/ls bin/systrace distrib/sets etc/etc.octeon/MAKEDEV etc/etc.octeon/MAKEDEV.md etc/root/root.mail etc/signify/openbsd-59-base.pub games/backgammon gnu/usr.bin/binutils-2.17 lib/libc lib/libcrypto lib/libssl regress/lib regress/sys regress/usr.bin regress/usr.sbin share/man share/misc sys/arch/alpha/alpha sys/arch/amd64/amd64 sys/arch/amd64/include sys/arch/amd64/stand/boot sys/arch/arm/include sys/arch/armish/include sys/arch/armish/stand/boot sys/arch/armv7/imx sys/arch/armv7/include sys/arch/armv7/omap sys/arch/armv7/sunxi sys/arch/hppa/stand/boot sys/arch/hppa64/stand/boot sys/arch/i386/i386 sys/arch/i386/include sys/arch/i386/isa sys/arch/i386/stand/boot sys/arch/landisk/stand/boot sys/arch/luna88k/cbus sys/arch/macppc/conf sys/arch/macppc/dev sys/arch/macppc/include sys/arch/macppc/macppc sys/arch/macppc/pci sys/arch/macppc/stand sys/arch/octeon/conf sys/arch/octeon/dev sys/arch/octeon/octeon sys/arch/powerpc/conf sys/arch/powerpc/include sys/arch/powerpc/powerpc sys/arch/sgi/dev sys/arch/sgi/hpc sys/arch/sgi/xbow sys/arch/socppc/dev sys/arch/socppc/stand/boot sys/arch/sparc/dev sys/arch/sparc64/dev sys/arch/sparc64/sparc64 sys/arch/vax/if sys/arch/zaurus/include sys/arch/zaurus/stand/zboot sys/dev sys/dev/acpi sys/dev/ic sys/dev/isa sys/dev/pci sys/dev/pcmcia sys/dev/rasops sys/dev/sbus sys/dev/usb sys/kern sys/net sys/net80211 sys/netinet sys/sys sys/uvm usr.bin/compress usr.bin/du usr.bin/grep usr.bin/mandoc usr.bin/netstat usr.bin/nm usr.bin/sed usr.bin/ssh usr.sbin/bgpd usr.sbin/dhcpd usr.sbin/httpd usr.sbin/npppd usr.sbin/pkg_add == bin =============================================================== 01/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/bin chmod ~ chmod.c > Put fts_close() where missing. > Not bugs in short-lived commands that call exit() -> _exit() immediately, > but for idempotency. > Originally found in ls(1) by Valgrind. Changes for other commands are > from deraadt@. Reviewed by me, tested in snapshots. > OK deraadt@ (uebayasi@) ed ~ ed.1 > theo buehler points out that posix specifies a prompt defined using -p > be reinstated when toggled off then on again; this seems to make sense, > but posix is not explicit. > we know that gnu ed and freebsd do reinstate the user defined prompt, but > we do not. no one has yet stepped up to change this, so i'm adding a note > to this effect until such a time as we do/ > i have not tweaked the text for -p or "P" to allow for an easy back out > if behaviour does change; (jmc@) ls ~ ls.c > Put fts_close() where missing. > Not bugs in short-lived commands that call exit() -> _exit() immediately, > but for idempotency. > Originally found in ls(1) by Valgrind. Changes for other commands are > from deraadt@. Reviewed by me, tested in snapshots. > OK deraadt@ (uebayasi@) systrace ~ register.c > Apply normal handling to atfd+path args to chflagsat(2) > Problem noted by patrick keshishian (sidster (at) boxsoft.com) > ok deraadt@ (guenther@) == distrib =========================================================== 02/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib sets ~ lists/base/mi > sync (deraadt@) ~ lists/man/mi > sync (deraadt@) ~ lists/comp/md.aviion ~ lists/comp/md.luna88k > sync (miod@) == etc =============================================================== 03/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/etc etc.octeon/MAKEDEV ~ etc.octeon/MAKEDEV > sync (deraadt@) ~ etc.octeon/MAKEDEV > regen (jmatthew@) etc.octeon/MAKEDEV.md ~ etc.octeon/MAKEDEV.md > add usb devices (jmatthew@) root/root.mail ~ root/root.mail > fix emacs pkg names (giovanni@) signify/openbsd-59-base.pub + signify/openbsd-59-base.pub > 5.9 base key (deraadt@) == games ============================================================= 04/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/games backgammon ~ backgammon/main.c ~ backgammon/move.c ~ common_source/save.c ~ common_source/subs.c > /home/otto/ok (otto@) == gnu =============================================================== 05/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu usr.bin/binutils-2.17 ~ bfd/elf32-arm.c > Avoid an assertion failure in elf32_arm_size_dynamic_sections() when > building a static binary out of PIE objects; similar to the logic > found on other platforms, for some reason arm was lacking this fix. (miod@) == lib =============================================================== 06/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libc ~ stdlib/merge.c > Just return if nmemb is 0. Avoids a NULL dereference and is > consistent with the behavior of the other libc sort functions. > OK deraadt@ (millert@) ~ gen/isblank.3 > Add STANDARDS section; isblank(3) was specified in C99. > ok jung@ jmc@ (bentley@) libcrypto ~ crypto/Makefile > Fix pointer to unsigned long conversion. > bcook@ notes that this check really only impacted 64-bit Windows. Also, > changed the check to be unsigned for consistency. > ok bcook@ (doug@) libssl ~ src/ssl/bs_ber.c > Check for failure with CBB_init() in bs_ber.c. > From BoringSSL commit 3fa65f0f05f67615d9daf48940e07f84d094ac6e. (doug@) ~ src/ssl/bs_cbs.c > Change CBS_dup() to also sync the offset. > Previously, CBS_dup() had its own offset. However, it is more consistent > to copy everything. > ok miod@ jsing@ (doug@) ~ src/ssl/bs_cbs.c ~ src/ssl/s3_clnt.c > Stop using BUF_memdup() within the LibreSSL code base - it is correctly > spelt malloc+memcpy, which is what is used in all except two places. > ok deraadt@ doug@ (jsing@) ~ src/crypto/buffer/buffer.h > Put BUF_memdup() and BUF_reverse() under #ifndef LIBRESSL_INTERNAL. > (jsing@) ~ src/crypto/cryptlib.c ~ ssl/Makefile > Fix pointer to unsigned long conversion. > bcook@ notes that this check really only impacted 64-bit Windows. Also, > changed the check to be unsigned for consistency. > ok bcook@ (doug@) ~ src/ssl/ssl_lib.c ~ src/ssl/ssl_locl.h > Convert ssl_bytes_to_cipher_list to CBS. > Link in the new 'unit' regress and expand the invalid tests to include > some that would fail before the CBS conversion. > input + ok miod@ jsing@ (doug@) == regress =========================================================== 07/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress lib ~ libssl/bytestring/Makefile > Remove unnecessary regress target. (doug@) ~ libssl/bytestring/bytestringtest.c > Convert bytestringtest to individual checks and don't short circuit. > The statements were chained together with OR which makes it more annoying > to debug. Also, it was short circuiting all tests as soon as one function > failed. Since the functions are independent, they should each run until > error. > Discussed with miod@ and jsing@ (doug@) ~ libssl/bytestring/bytestringtest.c > Change CBS_dup() to also sync the offset. > Previously, CBS_dup() had its own offset. However, it is more consistent > to copy everything. > ok miod@ jsing@ (doug@) + libssl/unit/Makefile + libssl/unit/cipher_list.c + libssl/unit/tests.h > Add unit tests for LibreSSL. > cipher_list.c is based on code from jsing@. > Discussed with jsing@ (doug@) ~ libssl/Makefile ~ libssl/unit/cipher_list.c > Convert ssl_bytes_to_cipher_list to CBS. > Link in the new 'unit' regress and expand the invalid tests to include > some that would fail before the CBS conversion. > input + ok miod@ jsing@ (doug@) sys ~ kern/sosplice/Remote.pm ~ net/pf_divert/Remote.pm > Align the three variants of Remote.pm. (bluhm@) usr.bin ~ sed/Makefile + sed/negation.expected > Add regress test for multiple negation characters. (millert@) ~ mandoc/roff/cond/if.in ~ mandoc/roff/cond/if.out_ascii > Ignore blank characters at the beginning of a conditional block, > that is, after "\{". > Issue found by Markus <Waldeck at gmx dot de> in bash(1). (schwarze@) usr.sbin + httpd/Makefile + httpd/patterns/Makefile + httpd/patterns/patterns-tester.c + httpd/patterns/patterns-tester.lua + httpd/patterns/test-patterns-lua.out + httpd/patterns/test-patterns.in + httpd/patterns/test-patterns.out > add regress tests for httpd > - this testsuite covers patterns (semarie@) ~ Makefile > add httpd regress (semarie@) ~ httpd/patterns/Makefile ~ httpd/patterns/test-patterns-lua.out > Fix the optional lua patterns test with obj and different versions. (reyk@) ~ relayd/Remote.pm > Align the three variants of Remote.pm. (bluhm@) ~ httpd/patterns/test-patterns.out > change test-patterns.out in order to reflect the error message change in > patterns.c (semarie@) ~ syslogd/funcs.pl ~ syslogd/syslogd.pl + syslogd/args-block-prog-quick.pl + syslogd/args-block-prog.pl > Add tests for the !prog and !!prog and !* keywords in syslog.conf. (bluhm@) ~ syslogd/args-client-udp.pl ~ syslogd/args-client-udp4.pl ~ syslogd/args-client-udp6.pl ~ syslogd/args-only4.pl ~ syslogd/args-only6.pl > Use fstat to check that the privsep parent of syslogd closes all > internet sockets also when invoked with -u. (bluhm@) == share ============================================================= 08/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man4/pipex.4 > Fix the links to pppx(4). > diff from Fabian Raetz. (yasuoka@) ~ man9/mbuf_tags.9 > PACKET_TAG_BRIDGE is gone. (mpi@) ~ man4/pfsync.4 > pfsync and ipsec do not currently work; from lukasz czarniecki > i've chosen to comment out the pertinent text rather than remove it... > confirmation/ok dlg (jmc@) ~ man4/upd.4 > Update upd(4): > - list new sensors, sorted by source file occurrence > - add clues about sensorsd.conf(5) configuration for the status quo > - link to HID Power specs > - linguistic tweaks > from David Higgs, ok+tweaks schwarze@ (sthen@) misc ~ mime.types > Add .mkv (video/x-matroska). > From David Hill > ok halex@ (reyk@) == sys =============================================================== 09/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/alpha/alpha ~ trap.c > delete more p==NULL checks; discussed with miod, kettenis, dlg before > (deraadt@) ~ locore.s > In the copy(9) function, make sure to remember curproc accross the bcopy() > call, instead of &curproc. The copy routine may sleep and we may resume on > a different processor. This has been plaguing the alpha MULTIPROCESSOR > kernels > since the very beginning; it's amazing this did not cause more havoc. > Joint debugging and hair pulling with dlg@ and deraadt@; ok dlg@ (miod@) ~ pmap.c > Disable some of the pmap_emulate_reference() DEBUG checks if option > MULTIPROCESSOR, and quote the alpha ARM to explain why; while there, make > the > failure messages a bit more detailed. (miod@) ~ debug.s > MP_LOCKDEBUG code should only be compiled if MULTIPROCESSOR is defined > too. (dlg@) arch/amd64/amd64 ~ mem.c > Make it possible to create write combing mappings through /dev/mem. This > is > done by introducining a magic offset. Pages below this offset are mapped > with default memory attributes. Above this offset pages are mapped write > combining. > ok mlarkin@ (kettenis@) ~ acpi_wakecode.S > Add an #ifdef HIBERNATE to allow to build a kernel without hibernate but > with acpi. > OK mlarkin@ (bluhm@) ~ locore.S > If the kernel symbols fit completely into the 2 MB alignment hole > after kernel bss but before end of the image, the page tables used > the read-only mapping of the hole. When booting a small non-generic > kernel, this resulted in a crash, while writing to the page tables > later. > Make sure that the page tables are created after esym and after > end. > OK mlarkin@ deraadt@ (bluhm@) ~ db_disasm.c > Uninitialized variables; pedro@ (miod@) ~ lock_machdep.c > you need to include ddb/db_output.h so you know how to call db_printf > inside MP_LOCKDEBUG. (dlg@) ~ db_trace.c ~ locore.S ~ spl.S ~ trap.c ~ vector.S > Split AST handling from trap() into ast() and get rid of T_ASTFLT. > Don't skip the AST check when returning from *fork() in the child. > Make sure to count interrupts even when they're deferred or stray. > testing by krw@, and then many via snapshots (guenther@) ~ locore.S ~ machdep.c ~ process_machdep.c ~ vm_machdep.c > Force the return to userspace from execve to go through iretq to get all > registers. This lets us kill the special handling of pid 1 in fork and > merge {proc,child}_trampoline(). Do the same if ptrace(PT_SETREGS) is used > to modify registers. > ok mlarkin@ kettenis@ (guenther@) arch/amd64/include ~ pmap.h > Remove a couple of unused and old #defines that discussed phys and virt > address widths in 1st-gen amd64 cpus. > ok kettenis, deraadt, guenther (mlarkin@) ~ trap.h > Split AST handling from trap() into ast() and get rid of T_ASTFLT. > Don't skip the AST check when returning from *fork() in the child. > Make sure to count interrupts even when they're deferred or stray. > testing by krw@, and then many via snapshots (guenther@) ~ segments.h > LDT is gone and not coming back (guenther@) ~ cpu.h > Force the return to userspace from execve to go through iretq to get all > registers. This lets us kill the special handling of pid 1 in fork and > merge {proc,child}_trampoline(). Do the same if ptrace(PT_SETREGS) is used > to modify registers. > ok mlarkin@ kettenis@ (guenther@) arch/amd64/stand/boot ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) arch/arm/include ~ pmap.h > Remove unused KERNEL_PD_SIZE macro which performs unsafe signed shift of > KERNEL_BASE. (miod@) arch/armish/include ~ vmparam.h > Make ARM_KERNEL_BASE explicitely unsigned (miod@) arch/armish/stand/boot ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) arch/armv7/imx ~ imxenet.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) arch/armv7/include ~ vmparam.h > Make ARM_KERNEL_BASE explicitely unsigned (miod@) arch/armv7/omap ~ if_cpsw.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) arch/armv7/sunxi ~ sxie.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) arch/hppa/stand/boot ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) arch/hppa64/stand/boot ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) arch/i386/i386 ~ mem.c > Make it possible to create write combing mappings through /dev/mem. This > is > done by introducining a magic offset. Pages below this offset are mapped > with default memory attributes. Above this offset pages are mapped write > combining. > ok mlarkin@ (kettenis@) ~ db_disasm.c > Uninitialized variables; pedro@ (miod@) ~ apicvec.s ~ db_trace.c ~ kgdb_machdep.c ~ locore.s ~ trap.c ~ vector.s > Split AST handling from trap() into ast() and get rid of T_ASTFLT > testing by krw@, and then many via snapshots (guenther@) arch/i386/include ~ trap.h > Split AST handling from trap() into ast() and get rid of T_ASTFLT > testing by krw@, and then many via snapshots (guenther@) arch/i386/isa ~ icu.s > Split AST handling from trap() into ast() and get rid of T_ASTFLT > testing by krw@, and then many via snapshots (guenther@) arch/i386/stand/boot ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) arch/landisk/stand/boot ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) arch/luna88k/cbus ~ nec86hw.c > Reimplement the audio driver in a simpler way, removing unused/unusable > functionality. Same API and ABI except for the removed bits and no > behaviour change for programs using libsndio. With help from armani@ > and mpi@, thanks. (ratchov@) arch/macppc/conf ~ files.macppc > move the ppc mplock implementation from macppc to powerpc. > ok mpi@ (dlg@) arch/macppc/dev ~ if_bm.c ~ if_mc.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) ~ adb.c ~ macgpio.c ~ macintr.c ~ openpic.c ~ xlights.c > IPL_MPSAFE bits for macppc with openpic(4). (mpi@) ~ openpic.c > Use a single event counter for IPIs like other archs do. > Even if this counter is subject to trashing it is a simple solution > and gives an proximation "good enough" of the number of IPIs on Quad > CPUs machine. > ok deraadt@ (mpi@) arch/macppc/include ~ mplock.h > move the ppc mplock implementation from macppc to powerpc. > ok mpi@ (dlg@) arch/macppc/macppc ~ ofw_machdep.c > Do not quiesce the firmware on Quad G5 to let it manage the fans. > This also unbreak "bsd -cd" on such machine since we use OpenFirmware's > input at this stage of the boot process. (mpi@) ~ ofw_machdep.c > Do not compare against a null string. When ofw_find_keyboard() is run > hw_prod is not initialized. > While here simply use "PowerBook" as model name, there's no "iBook" > model in Apple device-trees. > ok miod@ (who's cleaning one of his forests after 3 years) (mpi@) - lock_machdep.c > move the ppc mplock implementation from macppc to powerpc. > ok mpi@ (dlg@) arch/macppc/pci ~ ht.c > OF_getprop() returns an int, so comparing its return value against sizeof > casts > it to unsigned, and we need to also check for negative values. > All users of OF_getprop() did this but that one. > ok mpi@ (miod@) arch/macppc/stand ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) arch/octeon/conf ~ GENERIC ~ RAMDISK > enable octdwctwo and add umass (jmatthew@) arch/octeon/dev ~ cn30xxgmx.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) arch/octeon/octeon ~ machdep.c > Re-enable memory above 256mb now that uvm_pmr_get1page() has been fixed. > From Visa Hankala. (jmatthew@) arch/powerpc/conf ~ files.powerpc > move the ppc mplock implementation from macppc to powerpc. > ok mpi@ (dlg@) arch/powerpc/include ~ intr.h > IPL_MPSAFE bits for macppc with openpic(4). (mpi@) + mplock.h > move the ppc mplock implementation from macppc to powerpc. > ok mpi@ (dlg@) ~ mplock.h > rename the guard #define from _MACHINE_MPLOCK_H_ to _POWERPC_MPLOCK_H_ > (dlg@) ~ lock.h > remove __cpu_cas and use atomic_cas_ulong instead. > ok mpi@ (dlg@) arch/powerpc/powerpc + lock_machdep.c > move the ppc mplock implementation from macppc to powerpc. > ok mpi@ (dlg@) ~ lock_machdep.c > remove __cpu_cas and use atomic_cas_ulong instead. > ok mpi@ (dlg@) arch/sgi/dev ~ if_iec.c ~ if_mec.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) arch/sgi/hpc ~ if_sq.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) arch/sgi/xbow ~ xbridge.c > Attempt to explain how this device works. Based upon a private mail I wrote > 6 years ago when asked about this. (miod@) arch/socppc/dev ~ if_tsec.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) ~ ipic.c > IPL_MPSAFE bits for macppc with openpic(4). (mpi@) arch/socppc/stand/boot ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) arch/sparc/dev ~ be.c ~ hme.c ~ if_ie.c ~ qe.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) arch/sparc64/dev ~ vnet.c > Count transmitted packets. (kettenis@) ~ vnet.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) arch/sparc64/sparc64 ~ locore.s > fauls -> faults (miod@) ~ lock_machdep.c > fix format string in MP_LOCKDEBUG code (dlg@) arch/vax/if ~ if_de.c ~ sgec.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) ~ if_qe.c > Memory leak in attach if uballoc() fails; Brainy/Maxime Villard (miod@) ~ if_qe.c > Pass bus_space tag and handles to the QE_{WR,RD}CR macros, instead of > assuming > there's an `sc' local variable. This allows us to no longer have to fake a > softc at match time. (miod@) arch/zaurus/include ~ vmparam.h > Make ARM_KERNEL_BASE explicitely unsigned (miod@) arch/zaurus/stand/zboot ~ boot.8 > document that boot.conf can contain comments; > from tilo stritzky > thanks miod for help with the diff, and who also noted that > leading whitespace gets stripped too; (jmc@) dev ~ ipmi.c > memory leak on failure; from Maxime Villard (deraadt@) ~ audio.c ~ audio_if.h ~ mulaw.c ~ mulaw.h > Reimplement the audio driver in a simpler way, removing unused/unusable > functionality. Same API and ABI except for the removed bits and no > behaviour change for programs using libsndio. With help from armani@ > and mpi@, thanks. (ratchov@) dev/acpi ~ acpimadt.c > Completely skip entries for disabled LAPICs so they don't overwrite > legit enabled ones > problem reported by Pedro Caetano (pedrocaetano (at) binaryflows.com) > ok kettenis@ (guenther@) dev/ic ~ an.c > Don't use uninitialized data as a return value. > From Brainy via Maxime Villard via tech@. > ok kettenis@ (krw@) ~ aic6915.c ~ dc.c ~ dp8390.c ~ elink3.c ~ fxp.c ~ gem.c ~ hme.c ~ i82596.c ~ if_wi.c ~ lance.c ~ lemac.c ~ mtd8xx.c ~ re.c ~ rtl81x9.c ~ smc83c170.c ~ smc91cxx.c ~ ti.c ~ xl.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) ~ arcofi.c > Adapt to the removal of the symbolic input and output port names from > <sys/audioio.h> (miod@) dev/isa ~ if_ef_isapnp.c ~ if_eg.c ~ if_el.c ~ if_ex.c ~ if_ie.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) ~ ad1848.c ~ ess.c ~ sbdsp.c > Reimplement the audio driver in a simpler way, removing unused/unusable > functionality. Same API and ABI except for the removed bits and no > behaviour change for programs using libsndio. With help from armani@ > and mpi@, thanks. (ratchov@) ~ gus.c ~ gusvar.h > Cards with revision < 10 (qemu rev is 0), have no ad1848 chip and > don't attach the ad1848 driver. But the "malloc" method of gus_hw_if > is ad1848_malloc() which assumes a ad1848 is attached (and expect the > softc pointer to be a struct ad1848_softc, but get a gus_softc pointer > instead). Implement, the missing gus_{malloc,free,mappage,...} > routines. Add the missing mtx_{enter,leave} calls, as we're at it. > Found by mlarkin. > ok mlarkin (ratchov@) dev/pci ~ xhci_pci.c > Apparently some BIOSes not supporting xHCI natively switch USB ports > back to EHCI at suspend. So route the ports back to xHCI at resume. > Problem reported by Adam Wolk, thanks! (mpi@) ~ drm/i915/i915_gem.c > Flush out another small diff to reduce the diffs with Linux. (kettenis@) ~ drm/drm_linux.h ~ drm/i915/i915_drv.c ~ drm/i915/i915_drv.h ~ drm/i915/i915_gem.c ~ drm/i915/i915_irq.c ~ drm/i915/intel_display.c ~ drm/i915/intel_dp.c ~ drm/i915/intel_drv.h ~ drm/i915/intel_pm.c > Introduce Linux work queue APIs and use them. As a side-effect, this will > move some of the work from the system task queue to the driver-specific > task queue. > ok jsg@ (kettenis@) ~ if_age.c ~ if_alc.c ~ if_ale.c ~ if_bce.c ~ if_bge.c ~ if_bnx.c ~ if_cas.c ~ if_de.c ~ if_em.c ~ if_et.c ~ if_ix.c ~ if_ixgb.c ~ if_jme.c ~ if_lge.c ~ if_lii.c ~ if_msk.c ~ if_myx.c ~ if_nep.c ~ if_nfe.c ~ if_nge.c ~ if_oce.c ~ if_pcn.c ~ if_se.c ~ if_sis.c ~ if_sk.c ~ if_ste.c ~ if_stge.c ~ if_tht.c ~ if_tl.c ~ if_txp.c ~ if_vge.c ~ if_vic.c ~ if_vio.c ~ if_vmx.c ~ if_vr.c ~ if_vte.c ~ if_wb.c ~ if_xge.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) ~ drm/drm_linux.h ~ drm/i915/i915_drv.h ~ drm/i915/i915_gem.c ~ drm/i915/intel_display.c ~ drm/i915/intel_pm.c ~ drm/i915/intel_ringbuffer.c > Linux jiffies and OpenBSD ticks are the same thing. > ok jsg@ (kettenis@) ~ drm/drm_linux.h > #undef HZ before defining it such that compiling a kernel with -DHZ=xxx > works. (kettenis@) ~ envy.c > Reimplement the audio driver in a simpler way, removing unused/unusable > functionality. Same API and ABI except for the removed bits and no > behaviour change for programs using libsndio. With help from armani@ > and mpi@, thanks. (ratchov@) ~ if_de.c ~ if_devar.h > Ensure the setup block is DMA reachable. > tested by reyk, mlarkin, others (deraadt@) ~ if_de.c ~ if_devar.h > There is no excuse for using dma_alloc(9) when a bus_dma_tag_t is > available. > deraadt@ thinks this isn't worth it, but reyk@ and mlarkin@ tested it > anyway (kettenis@) ~ drm/drm_linux.h ~ drm/i915/i915_drv.c ~ drm/i915/i915_drv.h ~ drm/i915/i915_gem.c ~ drm/i915/i915_irq.c > Add Linux completion API and use it. > ok jsg@ (kettenis@) ~ drm/i915/intel_display.c > Read the lower 16-bits of the GCFGC "double word". Potentially fixes > issues > with some modes on machines with the 915GM chipset. > discussed with jsg@ (kettenis@) dev/pcmcia ~ if_cnw.c ~ if_malo.c ~ if_xe.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) dev/rasops ~ rasops.c > Don't leak mem if wsfont_rotate() fails. > Problem reported by Maxime Villard, ok miod@ (jca@) dev/sbus ~ be.c ~ qe.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) dev/usb ~ usb.h ~ xhci.c ~ uhub.c > Make xhci(4)'s root hub report the same status bits as physical USB3 hubs. > There's not bit to indicate the speed of a USB3.0 device attached to a hub > port so do not abuse the PORT_TEST bit. Instead make the xhci(4) root hub > report the PORT_POWER_SS bit when appropriate and use it to determin the > speed of a new device. > While here make the root hub report the link state and config error, from > FreeBSD. (mpi@) ~ ehci.c ~ ohci.c ~ uhci.c ~ dwc2/dwc2.c > Do not update frame lengths to reflect what has really been transfered > when an isochronous transfer is done. > Frame lengths are just input values and no driver mess with them. > ok ratchov@, jmatthew@ (mpi@) ~ if_atu.c ~ if_aue.c ~ if_axe.c ~ if_axen.c ~ if_cdce.c ~ if_cdcef.c ~ if_cue.c ~ if_kue.c ~ if_mos.c ~ if_smsc.c ~ if_udav.c ~ if_ugl.c ~ if_upl.c ~ if_url.c ~ if_urndis.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) ~ usbdevs > SuperTop IDE bridge from frantisek holop. (mpi@) ~ usbdevs.h ~ usbdevs_data.h > regen (mpi@) ~ umass_quirks.c > Quirk needed for SuperTop IDE bridge from frantisek holop. (mpi@) ~ uvideo.c > Allow uvideo_mmap_queue() to fail gracefully when the mmap queue is full; > found the hard way by sebastia@ four years ago, and I'd been sitting on > that > diff since. > The initial diff was more aggressive and would free the mmap queue upon > error, > but jakemsr@ had objections against this behaviour; this diff only fails > gracefully instead of panic'ing. (miod@) ~ ehci.c ~ ohci.c ~ uhci.c ~ dwc2/dwc2.c > Revert previous. uvideo checks the transfered length of every frame and > this break some devices. > As found the hardway by Hugo Sastre via sobrado@ (mpi@) ~ dwc2/dwc2.c ~ dwc2/dwc2.h ~ dwc2/dwc2_core.c ~ dwc2/dwc2_core.h ~ dwc2/dwc2_coreintr.c ~ dwc2/dwc2_hcd.c ~ dwc2/dwc2_hcd.h ~ dwc2/dwc2_hcdddma.c ~ dwc2/dwc2_hcdintr.c ~ dwc2/dwc2_hcdqueue.c > Convert list_head lists into TAILQs and LISTs and fix up header includes > so we can build dwc2 without extra stuff. > tested by several edgerouter lite owners, ok jasper@ (jmatthew@) kern ~ init_main.c > reenable the pool gc task. > the problems it tickled by working outside the biglock on archs > with mutex and clock interaction have been fixed, as evidenced by > the softnet taskq. > ok deraadt@ (dlg@) net ~ route.c > rtrequest1(9) error code path cleanup. > Pass the length to free(9), do not violate the radix/route layer and > set the gateway of a route a bit later to simplify error code path. > ok claudio@ (mpi@) ~ pf.c > Increment rule counters only after successful state insertion > Do rule counter increments after state has been successfully > installed. This has an additional benefit of making error > handling a bit simpler. > OK mpi, bluhm (mikeb@) ~ if.c ~ if_bridge.c ~ if_bridge.h ~ if_ethersubr.c ~ if_vlan.c > Adapt bridge(4) to the new if_input() framework. > Move bridge_input() outside of ether_input() in order to duplicate packets > flowing through a bridge port before applying any transformation on mbufs. > This saves a various m_adj(9)/M_PREPEND(9) dances and remove the bridge(4) > hack from vlan(4). > Tested by mxb <mxb AT alumni DOT chalmers DOT se> and kettenis@ > ok bluhm@ (mpi@) ~ if_ethersubr.c > Pass a "struct ifnet *" instead of a "struct arpcom *" to arpresolve(). > Most of the ARP layer already take an ifp pointer and this makes clear > wich chunks of code are messing with ac_enaddr. > Note that our Ethernet code assume that these pointer are interchangeable > since the first element of the "struct arpcom" is a "struct ifnet". (mpi@) ~ if.c ~ if_bridge.c ~ if_trunk.c ~ if_tun.c ~ if_vlan.c ~ if_vxlan.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) ~ if.c > Stop garbgage collecting mbufs from the ARP, IPv4 and IPv6 queues when > an interface is destroyed or removed. > Since the introduction of ph_ifidx, as soon as a mbuf with an invalid > interface index is dequeued it will be freed. > ok claudio@, dlg@ (mpi@) ~ if.c ~ if_ethersubr.c > Check for correct rdomain in if_output() rather than in ether_output(). > Not all drivers make use of ether_output() and there's no real reason to > call it when no ARP resolution is needed. But in this case we still want > to make sure we're sending packets in the correct rdomain. > ok bluhm@, claudio@ as part of a larger diff. (mpi@) ~ if.c ~ if_bridge.c ~ if_ethersubr.c > Move brige(4)'s output hook outside of ether_output(). > This fix some weird bridge(4) configurations involving pseudo-drivers > stacked on top of interfaces in a bridge. > Also simplifies the loop prevention logic to match bridge's input path. > Instead of using a tag per port/bridge simply flag output mbufs to make > sure only one copy per bridge go through bridge_output(). > ok bluhm@, claudio@ (mpi@) ~ if_ethersubr.c > No more NBRIDGE in the Ethernet layer. Fewer layer violation, say yeah! > (mpi@) ~ if_bridge.c > Properly deliver broadcast-like packets to the network stack. > In bridge(4) speak, broadcast-like packets are Ethernet Multicast > frames or Unicast for which the destination is unknown. > It makes sense to not retransmit broadcast-like packets on the interface > they were received but they still must be delivered to the network stack. > Problem reported by and ok jasper@ (mpi@) net80211 ~ ieee80211_input.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) netinet ~ ip_ether.c > Adapt bridge(4) to the new if_input() framework. > Move bridge_input() outside of ether_input() in order to duplicate packets > flowing through a bridge port before applying any transformation on mbufs. > This saves a various m_adj(9)/M_PREPEND(9) dances and remove the bridge(4) > hack from vlan(4). > Tested by mxb <mxb AT alumni DOT chalmers DOT se> and kettenis@ > ok bluhm@ (mpi@) ~ if_ether.c ~ if_ether.h > Pass a "struct ifnet *" instead of a "struct arpcom *" to arpresolve(). > Most of the ARP layer already take an ifp pointer and this makes clear > wich chunks of code are messing with ac_enaddr. > Note that our Ethernet code assume that these pointer are interchangeable > since the first element of the "struct arpcom" is a "struct ifnet". (mpi@) ~ ip_carp.c > Increment if_ipackets in if_input(). > Note that pseudo-drivers not using if_input() are not affected by this > conversion. > ok mikeb@, kettenis@, claudio@, dlg@ (mpi@) sys ~ memrange.h > Make it possible to create write combing mappings through /dev/mem. This > is > done by introducining a magic offset. Pages below this offset are mapped > with default memory attributes. Above this offset pages are mapped write > combining. > ok mlarkin@ (kettenis@) ~ audioio.h ~ conf.h > Reimplement the audio driver in a simpler way, removing unused/unusable > functionality. Same API and ABI except for the removed bits and no > behaviour change for programs using libsndio. With help from armani@ > and mpi@, thanks. (ratchov@) ~ mbuf.h > Move brige(4)'s output hook outside of ether_output(). > This fix some weird bridge(4) configurations involving pseudo-drivers > stacked on top of interfaces in a bridge. > Also simplifies the loop prevention logic to match bridge's input path. > Instead of using a tag per port/bridge simply flag output mbufs to make > sure only one copy per bridge go through bridge_output(). > ok bluhm@, claudio@ (mpi@) uvm ~ uvm_pmemrange.c > uvm_pmr_get1page() should return psize_t, not int; dhill@ (miod@) == usr.bin =========================================================== 10/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin compress ~ main.c > Put fts_close() where missing. > Not bugs in short-lived commands that call exit() -> _exit() immediately, > but for idempotency. > Originally found in ls(1) by Valgrind. Changes for other commands are > from deraadt@. Reviewed by me, tested in snapshots. > OK deraadt@ (uebayasi@) du ~ du.c > Put fts_close() where missing. > Not bugs in short-lived commands that call exit() -> _exit() immediately, > but for idempotency. > Originally found in ls(1) by Valgrind. Changes for other commands are > from deraadt@. Reviewed by me, tested in snapshots. > OK deraadt@ (uebayasi@) grep ~ util.c > Put fts_close() where missing. > Not bugs in short-lived commands that call exit() -> _exit() immediately, > but for idempotency. > Originally found in ls(1) by Valgrind. Changes for other commands are > from deraadt@. Reviewed by me, tested in snapshots. > OK deraadt@ (uebayasi@) mandoc ~ roff.c > Ignore blank characters at the beginning of a conditional block, > that is, after "\{". > Issue found by Markus <Waldeck at gmx dot de> in bash(1). (schwarze@) netstat ~ inet.c > There is no need to include sys/ucred.h. Only sys/file.h is needed for the > DTYPE defines. (claudio@) nm ~ elf.c > add some check before accessing data: > - if section header table is be present > - consistency of section header table size > ok miod@ (semarie@) ~ elf.c ~ elfuncs.h > corrects a read after bound that occurs in strcmp (line just > after the added bound check). > Found with afl. > ok miod@ (semarie@) ~ elf.c > This patch ensure that e_shentsize (sections header's size in bytes) is > big enough to fill at least one Elf_Shdr. > While here, inverts calloc() arguments to be calloc(nmemb, size), > according to fread() call after. > This problem was found with afl, with e_shentsize=1. > ok miod@ (semarie@) ~ elf.c ~ util.h > This patch ensure that when an error is detected, the freed variables in > elf_symloadx() are reinitialised. > Else show_file() in nm.c will used these variables, even if they has > just been freed. (nm.c +689). > Problem found by afl. > ok miod@ (semarie@) sed ~ compile.c > POSIX specifies that that multiple '!' characters preceding a > function should be treated as a single negation. From FreeBSD > via Liviu Daia. (millert@) ssh ~ uidswap.c > Don't call setgroups if we have zero groups; there's no guarantee that it > won't try to deref the pointer. Based on a patch from mail at > quitesimple.org, > ok djm deraadt (dtucker@) ~ monitor.c > Don't count successful partial authentication as failures in monitor; > this may have caused the monitor to refuse multiple authentications > that would otherwise have successfully completed; ok markus@ (djm@) ~ uidswap.c > Revert previous commit. We still want to call setgroups in the case where > there are zero groups to remove any that we might otherwise inherit (as > pointed out by grawity at gmail.com) and since the 2nd argument to > setgroups is always a static global it's always valid to dereference in > this case. ok deraadt@ djm@ (dtucker@) ~ krl.c > correct test to sshkey_sign(); spotted by Albert S. (djm@) ~ readconf.c > Fix \-escaping bug that caused forward path parsing to skip two > characters and skip past the end of the string. > Based on patch by Salvador Fandino; ok dtucker@ (djm@) == usr.sbin ========================================================== 11/11 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin bgpd ~ rde.c > There is a race between sending notifications to the SE and getting a new > peer_up event in the RDE. This can be triggered by graceful restart. So > remove the panic and replace it with roughly what peer_down does. > OK phessler and henning (claudio@) dhcpd ~ options.c ~ tables.c > Do not send routers (option 3) or static routes (option 33) when > classless static routes (option 121, 249) are sent. RFC 3442 says > servers SHOULD NOT send those options. > This avoids confusing clients who fail to ignore the routers or > static routes options in the presence of classless static routes > options. RFC 3442 says clients MUST ignore those options. > Incorrect behavior in MS and Linux clients reported by Juan on > misc@. > ok millert@ (krw@) httpd ~ server_http.c > When encoding the Location url, only encode the query and path > elements from the user input and not the constants from the > configuration. This makes it possible to specify chars like '?' in > the uri. > OK Sebastien Marie (reyk@) ~ server_http.c > After the last change, we also have to url_encode $SERVER_NAME and > $REMOTE_USER before using them in the Location. > From Sebastien Marie (semarie) (reyk@) ~ Makefile ~ httpd.conf.5 ~ httpd.h ~ parse.y ~ server_http.c + patterns.7 + patterns.c + patterns.h > Add initial support for pattern matching using Lua's pattern matching code. > With important help on the pattern matcher from semarie@ > OK semarie@ (reyk@) ~ patterns.c > remove a deprecated character class. > it was deprecated in lua code, but here the code is new. The documentation > don't mention it either. > ok reyk@ (semarie@) ~ server_http.c > escape the matched substrings before using it in expansion. > ok reyk@ (semarie@) ~ httpd.conf.5 ~ patterns.7 > various tweaks; (jmc@) ~ patterns.c > Corrects some minors nits. Patch from Theo Buehler. > - cleanup in included headers (removing unsed assert.h, and reorder) > - one remaining '%%' in an error string corrected in '%' > while here, add sys/types.h for off_t type. > OK reyk@ (semarie@) ~ patterns.h > move #include inside #ifndef PATTERNS_H > OK reyk@ (semarie@) ~ patterns.7 > Corrects the manpage for patterns(7): the indexing for empty capture follow > C-style (starting from 0) and not the Lua-style (starting from 1). > Patch from Theo Buehler. > OK reyk@ (semarie@) npppd ~ l2tp/l2tpd.c ~ pptp/pptpd.c > Fix the problem when npppd receives a zero length 1701/udp packet. If > it receives such packets when the errno is not EAGAIN or EINTR, it had > closed all L2TP sessions and stoppped the L2TP server. Also fix the > receiving GRE packet since it potentially has the same problem. (yasuoka@) ~ pptp/pptpd.c > Fix npppd to check the size of received GRE packets properly. (yasuoka@) ~ npppd/npppd.conf.5 > Mention that pipex should be also configured by sysctl. (yasuoka@) ~ npppd/chap.c ~ npppd/lcp.c > Improve and fix typo in the comment. (yasuoka@) ~ npppd/npppd.c > Use the return value of event_loop() properly. Also abort(3) if > event_loop() returns abnormally which is unexpected. > diff from Yuuichi Someya (yasuoka@) ~ l2tp/l2tp.h ~ l2tp/l2tp_call.c ~ l2tp/l2tp_ctrl.c > Fix npppd to terminate all PPP sessions properly even in case the > sending window for L2TP control is full when the control is terminating > (by a L2TP keepalive failure or other reasons). In that case, if the > L2TP peer didn't respond at all, npppd had kept some PPP sessions > forever. (yasuoka@) ~ npppd/npppd.conf.5 > tweak previous; (jmc@) ~ npppd/lcp.c > Reset the LCP option state for dialin-proxy only if re-negotiation is > enabled. (yasuoka@) ~ npppd/lcp.c > Fixed a use-after-free. When handing the L2TP Proxy-LCP, a timer was used > to open LCP. If the PPP session is terminated before the timer fires, the > timer had been fired with the freed context. Stop using the timer to open > LCP. (yasuoka@) pkg_add ~ OpenBSD/PackageRepository/Installed.pm > use machine_architecture (arch -s) when assembling PKG_PATHs using %m or > %a, > fixes these variables with arch like powerpc where this differs from "uname > -m". > reported by/ok aja@ (sthen@) =============================================================================== _______________________________________________ owc mailing list [email protected] http://www.squish.net/mailman/listinfo/owc
