OpenBSD src changes summary for 2015-09-09
==========================================

distrib/miniroot                        distrib/sets
gnu/usr.bin/binutils-2.17               gnu/usr.sbin/mkhybrid
include/Makefile                        include/mpool.h
include/unistd.h                        lib/csu
lib/libc                                lib/libcrypto
lib/libssl                              lib/libtls
libexec/ld.so                           regress/usr.sbin
sbin/disklabel                          sbin/fdisk
sbin/ping6                              share/man
sys/arch/mips64/conf                    sys/arch/mips64/include
sys/arch/mips64/mips64                  sys/arch/octeon/conf
sys/arch/octeon/include                 sys/arch/octeon/octeon
sys/arch/sgi/conf                       sys/arch/sgi/include
sys/arch/sgi/sgi                        sys/dev/ic
sys/dev/pci                             sys/isofs/cd9660
sys/kern                                sys/net
sys/netinet                             sys/netinet6
sys/sys                                 sys/uvm
usr.bin/at                              usr.bin/ftp
usr.bin/ipcrm                           usr.bin/mg
usr.bin/sendbug                         usr.bin/ssh
usr.bin/tmux                            usr.bin/uuencode
usr.bin/yacc                            usr.bin/ypwhich
usr.sbin/ntpd                           usr.sbin/syslogd
usr.sbin/sysmerge                       usr.sbin/ypbind
usr.sbin/ypldap                         usr.sbin/yppoll
usr.sbin/ypset                          

== distrib =========================================================== 01/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/distrib

miniroot

  ~ install.sub                           

  > At some point back in time, the disk info (obtained by responding '?' at
  > the root disk question) got some extra linefeeds in it. This fixes the
  > output to be one line per disk again.
  > ok krw@ (halex@)

  ~ install.sh                            ~ install.sub

  > zap trailing whitespace (halex@)

sets

  ~ lists/base/md.alpha                   ~ lists/base/md.amd64
  ~ lists/base/md.armish                  ~ lists/base/md.armv7
  ~ lists/base/md.aviion                  ~ lists/base/md.hppa
  ~ lists/base/md.hppa64                  ~ lists/base/md.i386
  ~ lists/base/md.landisk                 ~ lists/base/md.loongson
  ~ lists/base/md.luna88k                 ~ lists/base/md.macppc
  ~ lists/base/md.octeon                  ~ lists/base/md.sgi
  ~ lists/base/md.socppc                  ~ lists/base/md.sparc
  ~ lists/base/md.sparc64                 ~ lists/base/md.vax
  ~ lists/base/md.zaurus                  ~ lists/comp/mi

  > sync (deraadt@)

  ~ lists/base/md.alpha                   ~ lists/base/md.amd64
  ~ lists/base/md.armish                  ~ lists/base/md.armv7
  ~ lists/base/md.aviion                  ~ lists/base/md.hppa
  ~ lists/base/md.hppa64                  ~ lists/base/md.i386
  ~ lists/base/md.landisk                 ~ lists/base/md.loongson
  ~ lists/base/md.luna88k                 ~ lists/base/md.macppc
  ~ lists/base/md.octeon                  ~ lists/base/md.sgi
  ~ lists/base/md.socppc                  ~ lists/base/md.sparc
  ~ lists/base/md.sparc64                 ~ lists/base/md.zaurus
  ~ lists/comp/mi                         

  > sync (deraadt@)

== gnu =============================================================== 02/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/gnu

usr.bin/binutils-2.17

  ~ ld/emultempl/elf32.em                 

  > For truly static binaries, force .ctors, .dtors and .got to be read-only.
  > This will make the segment containing those sections read-only for binaries
  > that use the OpenBSD W^X layout, preventing W^X violations on architectures
  > that need an executable GOT (basically BSS-PLT powerpc).
  > ok miod@ (kettenis@)

usr.sbin/mkhybrid

  ~ mkhybrid/Makefile                     ~ src/eltorito.c
  ~ src/joliet.c                          ~ src/mkisofs.h
  ~ src/multi.c                           ~ src/name.c
  ~ src/tree.c                            ~ src/write.c

  > Fix various buffer overflows (caused by the way struct iso_directory_record
  > is defined), and make this work on unsigned char platforms by using
  > isascii() instead of < 0.
  > While there, do not use the local getopt() implementation, as libc has had
  > getopt_long() support for years now.
  > With help from jca@ and guenther@. ok millert@, deraadt@ (on an earlier
  > diff) (miod@)

== include =========================================================== 03/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/include

Makefile

  ~ Makefile                              

  > Stop exporting from libc the <mpool.h> and the mpool_* API
  > ports scan by sthen@ (guenther@)

mpool.h

  - mpool.h                               

  > Stop exporting from libc the <mpool.h> and the mpool_* API
  > ports scan by sthen@ (guenther@)

unistd.h

  ~ unistd.h                              

  > Move to next tame() API.  The flags are now passed as a very simple string,
  > which results in tame() code placements being much more recognizeable.
  > tame() can be moved to unistd.h and does not need cpp symbols to turn the
  > bits on and off.  The resulting API is a bit unexpected, but simplifies the
  > mapping to enabling bits in the kernel substantially.
  > vague ok's from various including guenther doug semarie (deraadt@)

== lib =============================================================== 04/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/lib

csu

  ~ powerpc/md_init.h                     

  > Now that ld(1) will generate a read-only GOT for truly static binaries, we
  > no longer have to use mprotect(2) to take away PROT_WRITE.  This fixes ld
  > -Z
  > and paves the way for the new Secure-PLT ABI.
  > ok miod@ (kettenis@)

  ~ powerpc/md_init.h                     

  > Fetch the _GLOBAL)OFFSET_TABLE_ and _DYNAMIC pointers in a way that's
  > compatible with the Secure-PLT ABI. (kettenis@)

libc

  ~ arch/powerpc/sys/cerror.S             

  > Backtrack a little bit and don't try to use the per-thread errno location
  > if
  > r2 is zero.  Some upcoming changes will probably change the offset at which
  > the errno pointer is stored, or might even store errno directly.  Instead,
  > call __errno to get the location of errno.  Make sure we go through the PLT
  > such that we call the version in libpthread if it is linked in.  Set up r30
  > for this purpose.  The ___cerrno function itself remains protected.
  > ok guenther@ (kettenis@)

  ~ sys/Makefile.inc                      

  > Simplify makefile rules for building the ASM stubs into "pick something
  > from column A and something from column B".
  > ok miod@ deraadt@ (guenther@)

  ~ gen/syslog_r.c                        

  > Syslogd can handle 8192 bytes per message now.  Increase the libc
  > syslog(3) limit to the same size.  The buffer needs one byte more
  > as it conains a NUL that sendsyslog(2) does not process.
  > OK deraadt@ (bluhm@)

  ~ Symbols.list                          + include/mpool.h

  > Stop exporting from libc the <mpool.h> and the mpool_* API
  > ports scan by sthen@ (guenther@)

  ~ Symbols.list                          + hidden/rpcsvc/yp.h

  > Hide YP-specific xdr_* functions, and have the YP tools link -lrpcsvc
  > as needed.
  > ok miod guenther (deraadt@)

  ~ asr/res_send_async.c                  ~ rpc/clnt_udp.c

  > Use SOCK_NONBLOCK in a couple more places
  > ok deraadt@ (guenther@)

  ~ Symbols.list                          ~ asr/asr.c
  ~ asr/asr_debug.c                       ~ asr/asr_private.h
  ~ asr/asr_utils.c                       ~ asr/getaddrinfo_async.c
  ~ asr/gethostnamadr_async.c             ~ asr/getnameinfo_async.c
  ~ asr/getnetnamadr_async.c              ~ asr/getrrsetbyname_async.c
  ~ asr/res_init.c                        ~ asr/res_mkquery.c
  ~ asr/res_search_async.c                ~ asr/res_send_async.c

  > Hide all unnecessary asr / resolver related API with _ prefixes.
  > direction & ok guenther (deraadt@)

  ~ gen/siginterrupt.c                    ~ gen/signal.c
  ~ hidden/signal.h                       ~ hidden/string.h
  ~ Symbols.list                          

  > Hide __sigintr and __strsignal as internal implementation details.
  > Delete the already hidden _signal alias too.
  > ok deraadt@ millert@ (guenther@)

  ~ shlib_version                         

  > Bump for symbol removals (guenther@)

  ~ db/Makefile.inc                       

  > Remove the mpool(3) manpages
  > prodded by miod@ (guenther@)

  ~ sys/tame.2                            

  > Move to next tame() API.  The flags are now passed as a very simple string,
  > which results in tame() code placements being much more recognizeable.
  > tame() can be moved to unistd.h and does not need cpp symbols to turn the
  > bits on and off.  The resulting API is a bit unexpected, but simplifies the
  > mapping to enabling bits in the kernel substantially.
  > vague ok's from various including guenther doug semarie (deraadt@)

  ~ sys/tame.2                            

  > seperate->separate;
  > while here, i've adjusted the formatting of one list (jmc@)

libcrypto

  ~ man/ASN1_OBJECT_new.3                 ~ man/ASN1_STRING_length.3
  ~ man/ASN1_STRING_new.3                 ~ man/ASN1_STRING_print_ex.3
  ~ man/ASN1_generate_nconf.3             ~ man/BF_set_key.3
  ~ man/BIO.3                             ~ man/BIO_ctrl.3
  ~ man/BIO_f_base64.3                    ~ man/BIO_f_buffer.3
  ~ man/BIO_f_cipher.3                    ~ man/BIO_f_md.3
  ~ man/BIO_f_null.3                      ~ man/BIO_find_type.3
  ~ man/BIO_new.3                         ~ man/BIO_new_CMS.3

  > Add missing CVS Mdocdate tags.  No text change, except for the change
  > of date that can't be helped. (schwarze@)

libssl

  ~ src/ssl/ssl_lib.c                     

  > Check handshake_func against NULL rather than 0, since it is a function
  > pointer.
  > ok bcook@ miod@ (jsing@)

libtls

  ~ tls_client.c                          ~ tls_server.c

  > Only take ownership of a socket if we allocated it within libtls. If we are
  > passed a socket then the caller is responsible for closing it.
  > ok bcook@ (jsing@)

  ~ tls.c                                 

  > nuke trailing whitespace
  > ok jsing@ (beck@)

  ~ tls.c                                 

  > always clear errno when coming back from tls_read tls_write, and tls_close.
  > this avoids the problem of people checking for return values < 0
  > and then checking for errno before checking for TLS_READ_AGAIN
  > TLS_WRITE_AGAIN - since we can not guarantee what errno will be
  > set to from the underlying library calls (beck@)

  ~ tls.c                                 ~ tls.h
  ~ tls_client.c                          ~ tls_config.c
  ~ tls_internal.h                        ~ tls_server.c

  > Add client certificate support. Still needs a few tweaks but this will
  > ride upcoming minor bump
  > ok jsing@ (beck@)

  ~ tls.c                                 ~ tls_client.c
  ~ tls_config.c                          ~ tls_server.c
  ~ tls_util.c                            ~ tls_verify.c

  > Indent labels with a space so that diff -p is more friendly.
  > Requested by bluhm@ (jsing@)

== libexec =========================================================== 05/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/libexec

ld.so

  ~ sparc/rtld_machine.c                  

  > Some explicit iflush instructions after the kbind() call are still needed
  > for now; misunderstanding between guenther@ and me. (miod@)

== regress =========================================================== 06/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/regress

usr.sbin

  ~ syslogd/Client.pm                     ~ syslogd/funcs.pl
  + syslogd/args-bufsize-sendsyslog.pl    + syslogd/args-bufsize-udp.pl
  + syslogd/args-bufsize-unix.pl          

  > Test that syslogd can process 8 full size messages from sendsyslog(2)
  > or UDP socket or UNIX domain socket at once. (bluhm@)

  - syslogd/args-sendsyslog-error.pl      - syslogd/args-sendsyslog.pl
  + syslogd/args-bufsize-native.pl        + syslogd/args-sendsyslog-dropped.pl
  + syslogd/args-sendsyslog-native.pl     

  > Add args-bufsize-native.pl that checks wether libc syslog(3) can
  > handle 8192 bytes messages.  Rename the sendsyslog(2) tests
  > consistently. (bluhm@)

== sbin ============================================================== 07/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sbin

disklabel

  ~ disklabel.c                           

  > No need to check !xxboot inside a 'if (!xxboot) ...' block.
  > ok otto@ deraadt@ (krw@)

fdisk

  ~ fdisk.8                               

  > remove 3 paragraphs of useless information at the start, and replace
  > with the simple facts a user needs.
  > ok kettenis krw (deraadt@)

  ~ fdisk.8                               

  > In the flag command description, mention that a bootable partition
  > is marked with '*'
  > due to comments from sthen and kettenis (deraadt@)

ping6

  ~ ping6.c                               

  > Cleanup the iov handling in ping6. Use the global iov instead of a local
  > version which is then added to a global struct msghdr. Issue found by
  > Michael McConville. OK florian@ tobias@ (claudio@)

== share ============================================================= 08/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/share

man

  ~ man9/Makefile                         + man9/srpl_rc_init.9

  > implement a singly linked list built with SRPs.
  > this allows us to build lists of things that can be followed by
  > multiple cpus.
  > ok mpi@ claudio@ (dlg@)

  ~ man9/srpl_rc_init.9                   

  > millert@ noticed an extra period. (dlg@)

  ~ man9/srpl_rc_init.9                   

  > no comma after last entry in NAME; (jmc@)

== sys =============================================================== 09/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/sys

arch/mips64/conf

  ~ files.mips64                          

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

arch/mips64/include

  + mplock.h                              

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

arch/mips64/mips64

  + lock_machdep.c                        

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

arch/octeon/conf

  ~ files.octeon                          

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

arch/octeon/include

  ~ mplock.h                              

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

arch/octeon/octeon

  - lock_machdep.c                        

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

arch/sgi/conf

  ~ files.sgi                             

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

arch/sgi/include

  ~ mplock.h                              

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

arch/sgi/sgi

  - lock_machdep.c                        

  > The mplock implementations on MP-enabled mips64 platforms, octeon and
  > sgi, are identical. Put one implementation in mips64 and drop the
  > platform-specific copies, to remove duplicated code.
  > ok miod@ (visa@)

dev/ic

  ~ sti.c                                 ~ vga.c

  > sizes for free(); ok sthen semarie (deraadt@)

  ~ sili.c                                ~ siop.c
  ~ ti.c                                  

  > sizes for free(); ok semarie (deraadt@)

  ~ ami.c                                 ~ ciss.c
  ~ malo.c                                ~ mfi.c
  ~ mpi.c                                 

  > sizes for free(); ok sthen (deraadt@)

  ~ rt2560.c                              ~ rt2661.c

  > sizes for free(); ok semarie (deraadt@)

dev/pci

  ~ agp.c                                 ~ agp_amd.c
  ~ agp_i810.c                            

  > sizes for free(); ok semarie (deraadt@)

isofs/cd9660

  ~ cd9660_vfsops.c                       

  > Nuke some extraneous whitespace. (krw@)

kern

  ~ kern_srp.c                            

  > implement a singly linked list built with SRPs.
  > this allows us to build lists of things that can be followed by
  > multiple cpus.
  > ok mpi@ claudio@ (dlg@)

  ~ init_sysent.c                         ~ syscalls.c

  > sync (deraadt@)

  ~ syscalls.master                       ~ kern_tame.c

  > Move to next tame() API.  The flags are now passed as a very simple string,
  > which results in tame() code placements being much more recognizeable.
  > tame() can be moved to unistd.h and does not need cpp symbols to turn the
  > bits on and off.  The resulting API is a bit unexpected, but simplifies the
  > mapping to enabling bits in the kernel substantially.
  > vague ok's from various including guenther doug semarie (deraadt@)

  ~ subr_disk.c                           

  > No need to set d_npartitions in readdoslabel() or readgptlabel().
  > It has already been initialized in the MD readdisklabel() routines
  > when they call initdisklabel().
  > ok deraadt@ (krw@)

net

  ~ bpf.c                                 ~ bpfdesc.h

  > convert bpf to using an srp list for the list of descriptors.
  > this replaces the hand rolled list. the code has always used hand
  > rolled lists, but that gets a bit cumbersome when theyre SRPs.
  > requested ages ago by mpi@ (dlg@)

  ~ if_bridge.c                           

  > Kill a couple of if_get()s only needed to increment per-ifp IPv6 stats.
  > We do not export those per-ifp statistics and they will soon all die.
  > "We're putting inet6 on a diet" claudio@
  > ok dlg@, mikeb@, claudio@ (mpi@)

  ~ if_media.h                            

  > Remove IFM_MAKEMODE. It is unused in base and ports and there doesn't seem
  > to be a way of feeding it with a valid argument without peeking into the
  > internals of ifmedia macros.
  > ok mpi miod deraadt (stsp@)

  ~ if_pflow.c                            

  > size for free()
  > OK semarie@, deraadt@ (florian@)

  ~ if.c                                  ~ if.h
  ~ if_var.h                              

  > introduce reference counts for interfaces (ie, struct ifnet *ifp).
  > if_get can get a reference to an ifp, but it never releases that
  > reference. this provides an if_put function that can be used to
  > decrement the refcount.
  > we cannot come up with a scheme for letting the network stack run on
  > one (or many) cpus while ioctls are pulling interfaces down on another
  > cpu without refcounts for the interfaces.
  > if_put is going in now so we can go through the stack and put the
  > necessary calls to it in, and then we'll backfill this implementation
  > to actually check the refcounts when the interface detaches.
  > ok mpi@ mikeb@ claudio@ (dlg@)

  ~ pf.c                                  

  > add if_put after the if_get.
  > ok claudio@ (dlg@)

  ~ if_vxlan.c                            

  > if_put after the if_get.
  > ok mpi@ claudio@ (dlg@)

  ~ if_mpw.c                              

  > if_put after you if_get
  > ok claudio@ (dlg@)

  ~ if_tun.c                              

  > if_put after if_get.
  > this is a bit funky cos we give up the ref while sleeping in tunread,
  > and have to get a new ref on wakeup.
  > ok claudio@ (dlg@)

netinet

  ~ if_ether.c                            

  > Fix English in comment. OK dlg@ (claudio@)

  ~ ipsec_input.c                         

  > Kill a couple of if_get()s only needed to increment per-ifp IPv6 stats.
  > We do not export those per-ifp statistics and they will soon all die.
  > "We're putting inet6 on a diet" claudio@
  > ok dlg@, mikeb@, claudio@ (mpi@)

  ~ icmp6.h                               

  > Kill icmp6_ifstat_inc() and associated per-ifp storage.
  > The SIOCGIFSTAT_ICMP6 is no longer supported.
  > ok dlg@, mikeb@, claudio@ (mpi@)

  ~ if_ether.c                            

  > Rewrite arpproxy() to no longer iterate on the global list of interfaces.
  > ok claudio@ (mpi@)

  ~ in_pcb.c                              

  > if_put after you if_get
  > ok mpi@ (dlg@)

  ~ ip_divert.c                           

  > if_put after if_get
  > ok mpi@ (dlg@)

  ~ ip_ipip.c                             

  > if_put after if_get
  > ok mpi@ (dlg@)

  ~ in.c                                  

  > gotta if_put after you if_get
  > ok mpi@ (dlg@)

netinet6

  ~ raw_ip6.c                             

  > Kill a couple of if_get()s only needed to increment per-ifp IPv6 stats.
  > We do not export those per-ifp statistics and they will soon all die.
  > "We're putting inet6 on a diet" claudio@
  > ok dlg@, mikeb@, claudio@ (mpi@)

  ~ frag6.c                               ~ ip6_forward.c

  > Kill some commented out in6_ifstat_inc(). (mpi@)

  ~ icmp6.c                               ~ in6.c
  ~ in6_var.h                             ~ mld6.c
  ~ nd6_nbr.c                             ~ nd6_rtr.c
  ~ raw_ip6.c                             

  > Kill icmp6_ifstat_inc() and associated per-ifp storage.
  > The SIOCGIFSTAT_ICMP6 is no longer supported.
  > ok dlg@, mikeb@, claudio@ (mpi@)

sys

  ~ srp.h                                 

  > implement a singly linked list built with SRPs.
  > this allows us to build lists of things that can be followed by
  > multiple cpus.
  > ok mpi@ claudio@ (dlg@)

  ~ syscall.h                             ~ syscallargs.h

  > sync (deraadt@)

  ~ tame.h                                

  > Move to next tame() API.  The flags are now passed as a very simple string,
  > which results in tame() code placements being much more recognizeable.
  > tame() can be moved to unistd.h and does not need cpp symbols to turn the
  > bits on and off.  The resulting API is a bit unexpected, but simplifies the
  > mapping to enabling bits in the kernel substantially.
  > vague ok's from various including guenther doug semarie (deraadt@)

uvm

  ~ uvm_map.c                             ~ uvm_fault.c
  ~ uvm_extern.h                          

  > All our pmap implementations provide pmap_resident_count(), so remove
  > #ifndef pmap_resident_count code paths. (miod@)

  ~ uvm_map.c                             ~ uvm_map.h

  > Add locking for interrupt-safe maps (those that have the VM_MAP_INTRSAFE
  > flag
  > set).  Since we cannot sleep in that case, use a mutex instead of an
  > rwlock.
  > This is ok as the more complex code paths in the uvm code are not entered
  > for interrupt-safe maps as paging isn't allowed in those maps.
  > ok tedu@ (kettenis@)

== usr.bin =========================================================== 10/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin

at

  ~ atq.1                                 ~ atrm.1

  > The .Nd macro is intended to provide "a one line description", so avoid
  > multiple lines of input, in particular, don't use macros below .Nd;
  > such bad style can confuse various apropos(1) tools.  All deleted
  > macros reappear later in the text, anyway, except for .Tn, which i'm
  > deleting outright.
  > Found while scanning the tree for good and bad practices in preparation
  > for my EuroBSD2015 mdoc(7) tutorial on Oct 1 in Stockholm. (schwarze@)

ftp

  ~ fetch.c                               

  > Make sure we check TLS_WRITE_AGAIN when calling tls_read() and if
  > tls_read() fails, print the tls_error() rather than just the return value.
  > ok beck@ (jsing@)

ipcrm

  ~ ipcrm.1                               

  > The .Nd macro is intended to provide "a one line description", so avoid
  > multiple lines of input, in particular, don't use macros below .Nd;
  > such bad style can confuse various apropos(1) tools.  All deleted
  > macros reappear later in the text, anyway, except for .Tn, which i'm
  > deleting outright.
  > Found while scanning the tree for good and bad practices in preparation
  > for my EuroBSD2015 mdoc(7) tutorial on Oct 1 in Stockholm. (schwarze@)

mg

  ~ mg.1                                  

  > mg doesn't support 'dired-do-delete'. Change description to the
  > supported function. (lum@)

sendbug

  ~ sendbug.1                             

  > The .Nd macro is intended to provide "a one line description", so avoid
  > multiple lines of input, in particular, don't use macros below .Nd;
  > such bad style can confuse various apropos(1) tools.  All deleted
  > macros reappear later in the text, anyway, except for .Tn, which i'm
  > deleting outright.
  > Found while scanning the tree for good and bad practices in preparation
  > for my EuroBSD2015 mdoc(7) tutorial on Oct 1 in Stockholm. (schwarze@)

ssh

  ~ ssh-rsa.c                             

  > openssh_RSA_verify return type is int, so don't make it size_t within
  > the function itself with only negative numbers or zero assigned to it.
  > bz#2460 (djm@)

tmux

  ~ cfg.c                                 ~ client.c

  > No need to keep global options around for client which doesn't use them.
  > (nicm@)

uuencode

  ~ uuencode.1                            

  > fix a mistake in the predefined string passed to -width (it is
  > obvious "indent", not "ident", was intended); instead of this use
  > the macro Ds as the indent is an offset -- fixes indentation
  > of uuencode(1) and uudecode(1) options.
  > ok jmc@ (sobrado@)

yacc

  ~ yacc.1                                

  > The .Nd macro is intended to provide "a one line description", so avoid
  > multiple lines of input, in particular, don't use macros below .Nd;
  > such bad style can confuse various apropos(1) tools.  All deleted
  > macros reappear later in the text, anyway, except for .Tn, which i'm
  > deleting outright.
  > Found while scanning the tree for good and bad practices in preparation
  > for my EuroBSD2015 mdoc(7) tutorial on Oct 1 in Stockholm. (schwarze@)

ypwhich

  ~ Makefile                              

  > Hide YP-specific xdr_* functions, and have the YP tools link -lrpcsvc
  > as needed.
  > ok miod guenther (deraadt@)

== usr.sbin ========================================================== 11/11 ==

  http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin

ntpd

  ~ constraint.c                          

  > Fix memory leak in error path when max length exceeded.
  > From Michael McConville (millert@)

syslogd

  ~ syslogd.c                             

  > To double the receive buffer of a socketpair does not help as sending
  > checks the send buffer size.  So double both buffer sizes.
  > Moreover the default for sending is 2048 and for receiving is 4096.
  > This makes the existing double buffer algorithm inconsistent.  It
  > is better to make the buffers large enough to hold 8 full length
  > messages.  Just make sure that it does not shrink.
  > Keep the approach that doubles the buffer sizes.  When we are low
  > on buffers and cannot reach the full size, increase it as much as
  > possible.
  > While there, add consistent error messages.
  > OK benno@ (bluhm@)

sysmerge

  ~ sysmerge.sh                           

  > Move and tweak a comment. (ajacoutot@)

  ~ sysmerge.sh                           

  > Remove unused code. (ajacoutot@)

  ~ sysmerge.sh                           

  > Remove more dead code. (ajacoutot@)

ypbind

  ~ Makefile                              

  > Hide YP-specific xdr_* functions, and have the YP tools link -lrpcsvc
  > as needed.
  > ok miod guenther (deraadt@)

ypldap

  ~ Makefile                              

  > Hide YP-specific xdr_* functions, and have the YP tools link -lrpcsvc
  > as needed.
  > ok miod guenther (deraadt@)

yppoll

  ~ Makefile                              

  > Hide YP-specific xdr_* functions, and have the YP tools link -lrpcsvc
  > as needed.
  > ok miod guenther (deraadt@)

ypset

  ~ Makefile                              

  > Hide YP-specific xdr_* functions, and have the YP tools link -lrpcsvc
  > as needed.
  > ok miod guenther (deraadt@)

===============================================================================
_______________________________________________
odc mailing list
[email protected]
http://www.squish.net/mailman/listinfo/odc

Reply via email to