OpenBSD src changes summary for 2016-08-30 ==========================================
lib/libfuse lib/libutil libexec/ftpd libexec/ld.so regress/usr.bin sbin/ping sbin/ping6 share/man sys/arch/amd64/conf sys/arch/amd64/stand/efiboot sys/arch/loongson/loongson sys/arch/macppc/dev sys/dev/isa sys/kern sys/miscfs/fuse sys/msdosfs sys/net sys/nfs sys/sys usr.bin/mandoc usr.bin/openssl usr.bin/ssh usr.sbin/cron usr.sbin/httpd usr.sbin/smtpd usr.sbin/switchd == lib =============================================================== 01/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib libfuse ~ fuse_ops.c ~ fuse_private.h > Use struct stat for storing attributes in fusebufs, because using struct > vattr in userspace is suboptimal as some related helpers are not > available, e.g. VATTR_NULL() and IFTOVT(). The conversion is now done in > the kernel where it belongs. As a side effect the <sys/vnode.h> include > can be removed from libfuse. > tweaks and ok guenther (natano@) libutil ~ logwtmp.c ~ pty.c ~ readlabel.c ~ uucplock.c > Use O_CLOEXEC when opening fds local to a function > ok jca@ krw@ (guenther@) ~ uucplock.c > Use a constant format string and output the variable part with %s > ok krw@ millert@ (guenther@) == libexec =========================================================== 02/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/libexec ftpd ~ ftpd.c > remove ifdef for all the features we have. > retain disabled ip4in6 code until its future is decided. > ok deraadt jca (tedu@) ld.so ~ resolve.h > Move DT_NUM from <sys/exec_elf.h> to resolv.h in ld.so where it belongs. > ok guenther@ (kettenis@) == regress =========================================================== 03/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress usr.bin ~ mandoc/db/dbm_dump/dbm_dump.c ~ mandoc/db/out/all.derr ~ mandoc/db/out/all.mout ~ mandoc/db/run/Makefile + mandoc/db/out/all.merr + mandoc/db/out/badarch.dout + mandoc/db/out/badarch0.dout + mandoc/db/out/baddesc.dout + mandoc/db/out/baddesc0.dout + mandoc/db/out/badfile.dout + mandoc/db/out/badfile0.dout + mandoc/db/out/badname.dout + mandoc/db/out/badname0.dout + mandoc/db/out/badsect.dout + mandoc/db/out/badsect0.dout + mandoc/db/out/threearch.dout > Test handling of invalid pointers in the pages table. > In such cases, do not access NULL pointers in dbm_dump. > Make the setup in the run/Makefile a bit more systematic. > Output starts getting too long, so make everything silent; > you can always use "make -dl" to debug specific tests. (schwarze@) == sbin ============================================================== 04/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin ping ~ ping.c > usage() is __dead. > While here remove one useless call to usage() that's covered by the > very next line. (florian@) ~ ping.c > Avoid calling summary() from a signal handler. This will allow us > to go back to using stdio there. OK florian@ deraadt@ (millert@) ~ ping.c > summary() is no longer called from a signal handler, so it can use > stdio and does not need the workarounds. > ok florian millert (deraadt@) ping6 ~ ping6.c > Add destination IP to MAC, reduces difference to ping(8). (florian@) ~ ping6.c > usage() is __dead. > While here remove one useless call to usage() that's covered by the > very next line. (florian@) ~ ping6.c > Make sure to not decrease packet size when running with -v. > OK benno (florian@) ~ ping6.c > Avoid calling summary() from a signal handler. This will allow us > to go back to using stdio there. OK florian@ deraadt@ (millert@) ~ ping6.c > summary() is no longer called from a signal handler, so it can use > stdio and does not need the workarounds. > ok florian millert (deraadt@) == share ============================================================= 05/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share man ~ man4/pfsync.4 > use a mixture of .Dl and .Bd -compact to reduce the amount of > vertical whitespace, making it easier to see which hostname.if file > relates to which example; (jmc@) ~ man9/fb_setup.9 > Use struct stat for storing attributes in fusebufs, because using struct > vattr in userspace is suboptimal as some related helpers are not > available, e.g. VATTR_NULL() and IFTOVT(). The conversion is now done in > the kernel where it belongs. As a side effect the <sys/vnode.h> include > can be removed from libfuse. > tweaks and ok guenther (natano@) == sys =============================================================== 06/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys arch/amd64/conf ~ files.amd64 > Enable raspos24 for efifb. This makes qemu with UEFI start working. > ok deraadt (yasuoka@) arch/amd64/stand/efiboot ~ efiboot.c > Use MaxMode to terminate the mode iteration. Actually the old way > couldn't interate the ConsOut modes properly on qemu. > input NONAKA Kimihiro (yasuoka@) arch/loongson/loongson ~ generic2e_machdep.c > Mask sb pci irq bits for a,b,c,d instead of a,b,b,d. > Couldn't find someone with a loongson 2e to test this on hardware. > "looks ok" visa@ (jsg@) arch/macppc/dev ~ i2s.c > Properly set "msb" and "bps" fields for both play and rec directions. > Fixes "audio0: different play and record parameters ... " errors, and > probably other unreported errors. > help and ok landry@ (ratchov@) dev/isa ~ sch311x.c > Make the init code match the comment and disable watchdog > reset by mouse and keyboard instead of mouse and mouse. (jsg@) kern ~ sys_pipe.c ~ sysv_msg.c ~ sysv_shm.c > pool_setipl > ok natano@ (dlg@) miscfs/fuse ~ fuse_device.c ~ fuse_file.c ~ fuse_lookup.c ~ fuse_vfsops.c ~ fuse_vnops.c ~ fusebuf.c > Use struct stat for storing attributes in fusebufs, because using struct > vattr in userspace is suboptimal as some related helpers are not > available, e.g. VATTR_NULL() and IFTOVT(). The conversion is now done in > the kernel where it belongs. As a side effect the <sys/vnode.h> include > can be removed from libfuse. > tweaks and ok guenther (natano@) msdosfs ~ denode.h ~ msdosfs_vnops.c > Revert "Implement VFS read clustering for MSDOSFS" > This caused garbage to be written instead of blocks of 0-bytes if a > file is extended by seeking past the end. This happens for example > when extracting files containing lots of 0-bytes with tar. > ok mpi@ > Details of original commit: > msdosfs_vnops.c revision 1.105 > denode.h revision 1.28 > date: 2016/01/13 10:00:55; author: mpi; commitid: ru9jHQwQX09BC5Bw; > Implement VFS read clustering for MSDOSFS. > The logic used in msdosfs_bmap() to loop calling pcbmap() comes from > FreeBSD and is not really efficient but it is good enough since it is > only called when generating I/O. > With this diff I get a 100% improvement when reading big files from a > crappy USB stick. > With this and bread_cluster(9) modified to not re-fetch B_CACHED buffers, > reading large contiguous files with chunk sizes of MAXPHYS is almost as > fast as physio(9) on the same device. > For a 'real world' example, when copying music files from a USB stick I > see a speed jump from 15MB/s on -current to 24Mb/s with this diff. > While here rename some 'lbn' variables into 'cn' to better reflect what > we're dealing with. > Tested by Mathieu, with support from deraadt@ (sf@) net ~ art.c ~ art.h ~ rtable.c > use a per-table rwlock to serialize ART updates and walks, rather than > taking the kernel lock. > ok mpi@ dlg@ (jmatthew@) ~ pipex.c > pool_setipl > ok yasuoka@ (dlg@) ~ radix.c ~ route.c > pool_setipl > ok claudio@ mpi@ (dlg@) ~ pfkeyv2.c > pool_setipl > ok markus@ (dlg@) nfs ~ nfs_subs.c ~ nfs_syscalls.c > pool_setipl for nfs pools > ok natano@ (dlg@) sys ~ exec_elf.h > Move DT_NUM from <sys/exec_elf.h> to resolv.h in ld.so where it belongs. > ok guenther@ (kettenis@) ~ fusebuf.h > Use struct stat for storing attributes in fusebufs, because using struct > vattr in userspace is suboptimal as some related helpers are not > available, e.g. VATTR_NULL() and IFTOVT(). The conversion is now done in > the kernel where it belongs. As a side effect the <sys/vnode.h> include > can be removed from libfuse. > tweaks and ok guenther (natano@) == usr.bin =========================================================== 07/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin mandoc ~ dbm_map.c > less confusing warning message about negative offsets (schwarze@) ~ dbm.c > When the database is corrupt in the sense of containing invalid > pointers in the pages table, do not access NULL pointers, but > gracefully handle the errors. > Similar patches will be needed for the macro tables, too. > <attila at stalphonsos dot com> audited the code and pointed out to me > that dbm_get() can return NULL for corrupted databases, but that isn't > handled properly at various places. (schwarze@) openssl ~ openssl.1 > shorten rsautl; (jmc@) ~ s_time.c ~ apps.h > Fix 32-bit time handling, using time_t and make it work on systems > where that is long long. > ok beck guenther (deraadt@) ~ ca.c > buf[][] with strange use all over the place is ridiculous, especially > if buf[1] is never used. > ok guenther beck (deraadt@) ~ apps.c ~ apps.h ~ s_time.c > Add OPTION_ARG_TIME for parsing a (64 bit if needed) time_t > prodding & ok jsing (deraadt@) ssh ~ monitor.c > restrict monitor auth calls to be allowed only when their > respective authentication methods are enabled in the configuration. > prompted by Solar Designer; ok markus dtucker (djm@) == usr.sbin ========================================================== 08/08 == http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin cron ~ user.c > Fix fd leak on error. OK jsg@ (millert@) httpd ~ httpd.h ~ server_fcgi.c > Do not assume that the full http response header is in the first > fastcgi stdout record. Keep processing stdout records until we found > the header / body separator and only then generate the header > response. > Problem reported by many. > OK jung@ (florian@) ~ httpd.h > Remove duplicated prototypes from header. > "Looks good to me" natano@ (rzalamena@) ~ httpd.c ~ proc.c > Terminate daemon using the socket status instead of watching SIGCHLD or > kill()ing child process. > "Looks good to me" millert@ > ok benno@ (rzalamena@) ~ httpd.h ~ logger.c ~ proc.c ~ server.c > Kill (remove) the ps_pid from privsep struct since it is not being used > anymore. Also fix the process initialization prototypes. > ok reyk@ (rzalamena@) smtpd ~ parse.y > when configuring the daemon, assign values to the structure being conf-ed, > not the global structure. this worked by accident. > ok eric@ (gilles@) switchd ~ imsg_util.c > does not need sys/param.h (deraadt@) =============================================================================== _______________________________________________ odc mailing list [email protected] http://www.squish.net/mailman/listinfo/odc
