[odc] Daily src changes for 2007-06-28

[ODC]

OpenBSD src changes summary for 2007-06-28
==========================================

libexec/identd                          usr.bin/cvs
usr.bin/netstat                         usr.bin/rcs
usr.bin/sdiff                           usr.sbin/config

== libexec =========================================================== 01/03 ==

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

identd

  ~ identd.c                              

  > Add a SIGCHLD handler for background mode instead of doing waitpid()
  > at a single point in the event loop (which would always leave
  > a single zombie around after the first connection).
  > Also use daemon() instead of doing it by hand.  OK deraadt@ beck@
  > (millert@)

== usr.bin =========================================================== 02/03 ==

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

cvs

  ~ history.c                             

  > when logging to CVSROOT/history as a server fill in the
  > directory as <remote> instead of /tmp/cvs-serv<pid> (joris@)

  ~ update.c                              

  > do not call cvs_checkout_file() for file removals in local mode,
  > only required in server mode due to the fact we need to let the client
  > now its been removed, found this the hard way. (joris@)

  ~ commit.c                              

  > When commiting to a server in a remote setup classify
  > the files based upon their entry in CVS/Entries instead of
  > using the local classifying code and trying to locate the RCSfile
  > on our local disks, which ofc does not work correctly.
  > OK niallo@ (joris@)

  ~ add.c                                 ~ admin.c
  ~ checkout.c                            ~ client.c
  ~ cvs.h                                 ~ diff.c
  ~ diff3.c                               ~ edit.c
  ~ file.c                                ~ getlog.c
  ~ history.c                             ~ import.c
  ~ rcs.c                                 ~ remove.c
  ~ status.c                              ~ tag.c
  ~ update.c                              

  > Sync revisions and time buffers size to be consistent with each others.
  > Simplifies further size tweaks if needed.
  > OK niallo@ ray at . (xsa@)

netstat

  ~ main.c                                

  > Bring the index defines back in sync with struct nlist after removal of IPX
  > symbols from the table. This fixes the netstat issues some people reported.
  > OK henning@ (claudio@)

rcs

  ~ diff.c                                

  > Some syncs with diff(1):
  > - prepare() only returns 0, so change it to void and remove code
  > that checks its return value.
  > - Rename diff_len[] to len[].
  > - Remove casts that made the code a no-op:
  > (size_t)filesize <= SIZE_MAX
  > OK xsa (ray@)

  ~ diff.c                                

  > diff(1) sync, mostly spacing and moving crap around, except:
  > - stone() and newcand() cannot return < 0, so don't check it.
  > - \r, \v, \f are also whitespace
  > OK joris (ray@)

sdiff

  ~ sdiff.1                               ~ sdiff.c

  > Alias `1' and `2' to `l' and `r', making interactive merges easier
  > on the fingers.  This was recently added to GNU sdiff:
  > http://www.nabble.com/sdiff-UI-enhancement-t3877253.html
  > Initial diff from Steven Kreuzer <skreuzer (at) exit2shell dot com>.
  > OK jmc, millert, sobrado, deraadt. (ray@)

  ~ sdiff.1                               

  > groff understands `|', so no need to duplicate macros. (jmc@)

== usr.sbin ========================================================== 03/03 ==

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

config

  ~ ukc.c                                 

  > Add check for nlist failure to prevent a segfault.
  > Remove ret variable, which is only set but not never checked.
  > return (1) instead of exit(1) for consistency.
  > Original diff from Rainer Giedat.
  > OK millert, maja, and otto. (ray@)

===============================================================================