[odc] Daily src changes for 2006-03-17

ODC auto at squish.net
Sat Mar 18 07:00:28 GMT 2006


OpenBSD src changes summary for 2006-03-17
==========================================

bin/cp                                  bin/csh
bin/ksh                                 distrib/sgi
libexec/talkd                           regress/usr.bin
sbin/ping6                              sys/arch/alpha/include
sys/arch/cats/pci                       sys/dev/ic
sys/kern                                usr.bin/cvs
usr.bin/rcs                             usr.bin/ssh
usr.bin/tip                             usr.sbin/authpf
usr.sbin/config                         usr.sbin/pkg_add
usr.sbin/rdconfig                       

== bin =============================================================== 01/08 ==

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

cp

  ~ cp.c                                  

  > Call fts_close(); from NetBSD's Coverity analysis; ok jaredy@ millert@
  > (otto@)

csh

  ~ error.c                               

  > off-by-one; inspired by a netbsd coverity fix; ok deraadt@ (otto@)

ksh

  ~ eval.c                                ~ exec.c
  ~ history.c                             ~ io.c
  ~ proto.h                               

  > Simplify savefd() by removing the "noclose" flag and make noclose
  > behavior the default.  Almost all uses of savefd() are followed
  > by an implicit or explicit close.  OK otto@ (millert@)

== distrib =========================================================== 02/08 ==

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

sgi

  ~ ramdisk/install.md                    

  > Better (maybe even comprehensible) message about volume header and boot
  > loader. And fix a typo as well! Prompted by Jasper Lievisse Adriaanse.
  > ok miod@ (otto@)

== libexec =========================================================== 03/08 ==

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

talkd

  ~ print.c                               

  > fix a few off-by-ones; from NetBSD; ok deraadt@ (otto@)

== regress =========================================================== 04/08 ==

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

usr.bin

  ~ rcs/Makefile                          

  > cleanup; OK ray at .sa@)

== sbin ============================================================== 05/08 ==

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

ping6

  ~ ping6.c                               

  > Off-by-one; from NetBSD's coverity analysis; ok millert@ (otto@)

== sys =============================================================== 06/08 ==

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

arch/alpha/include

  ~ proc.h                                

  > Fix single-stepping via ptrace(2); ok art@ kettenis@ (miod@)

arch/cats/pci

  ~ pcib.c                                

  > Do not print the result of pci_devinfo() when attaching pcib, since this
  > has already be printed by the pci code. (miod@)

dev/ic

  ~ ami.c                                 ~ amivar.h

  > remove the last remnants of AMI_POLLING (dlg@)

  ~ ami.c                                 ~ amivar.h

  > split the completion path up. its now the responsibility of the the path
  > that sets the command up to say how its should be completed. this means
  > we dont have to complicate a generic handler to cope with all the different
  > setup paths. however, at the moment we're using the generic complicated
  > one :) (dlg@)

  ~ ami.c                                 

  > create a completion path for scsi commands without the cruft for waking
  > up ioctl paths. (dlg@)

kern

  ~ uipc_mbuf2.c                          

  > In m_pulldown avoid a prepend to the next mbuf in the chain if the result
  > would still not have all data we want continous.
  > From martin NetBSD
  > ok claudio@ mcbride@ (brad@)

  ~ uipc_mbuf.c                           

  > rev 1.77
  > m_cat() - if it is safe, copy data portion into 1st mbuf even if 1st mbuf
  > is M_EXT mbuf.
  > rev 1.72
  > clarify comment on m_cat().
  > From itojun NetBSD
  > ok claudio@ mcbride@ (brad@)

== usr.bin =========================================================== 07/08 ==

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

cvs

  ~ buf.c                                 ~ buf.h

  > Use ssize_t in appropriate places.
  > Found by lint.
  > OK niallo@ (ray@)

  ~ util.c                                ~ util.h

  > rlog_strsplit() -> cvs_strsplit() and move it to util.c so it can be reused
  > by `cvs log' and some other RCS utilities. "Looks good" ray at .sa@)

  ~ util.h                                ~ util.c

  > - Lint says cvs_hacktime() is not used in rcs, so move it into !RCSPROG.
  > ok xsa@ (niallo@)

rcs

  ~ ci.c                                  

  > Remove unused variables len and i, change `end' to size_t type.
  > Found by lint.
  > OK niallo@ (ray@)

  ~ rlog.c                                

  > rlog_strsplit() -> cvs_strsplit() and move it to util.c so it can be reused
  > by `cvs log' and some other RCS utilities. "Looks good" ray at .sa@)

  ~ co.c                                  

  > - co usage() says we support -I, so set the flags and revision.
  > ok xsa@ (niallo@)

ssh

  ~ authfd.c                              

  > unreachanble statement, found by lint (djm@)

  ~ authfd.c                              

  > another unreachable found by lint (djm@)

tip

  ~ acu.c                                 ~ cmds.c
  ~ cmdtab.c                              ~ cu.c
  ~ hunt.c                                ~ remote.c
  ~ tip.c                                 ~ tip.h
  ~ tipout.c                              ~ value.c
  ~ aculib/biz22.c                        ~ aculib/courier.c
  ~ aculib/df.c                           ~ aculib/hayes.c
  ~ aculib/t3000.c                        ~ aculib/ventel.c

  > big prototype cleanup:
  > o make more functions static
  > o add all the prototypes to tip.h
  > o consistently declare functions in the cmd table
  > testing and ok dhill@ (moritz@)

  ~ tipout.c                              

  > ARGSUSED is good on signal handlers (deraadt@)

  ~ tip.h                                 

  > global not needed; ok moritz (deraadt@)

  ~ acutab.c                              ~ tip.h
  ~ aculib/biz22.c                        ~ aculib/biz31.c
  ~ aculib/courier.c                      ~ aculib/df.c
  ~ aculib/dn11.c                         ~ aculib/hayes.c
  ~ aculib/t3000.c                        ~ aculib/v3451.c
  ~ aculib/v831.c                         ~ aculib/ventel.c

  > ansification and correct prototypes. ok deraadt@ (moritz@)

  ~ cmds.c                                ~ hunt.c
  ~ tip.c                                 ~ tip.h
  ~ tipout.c                              

  > more cleanups; ok moritz (deraadt@)

  ~ tipout.c                              

  > handle ssize_t vs size_t (deraadt@)

  ~ tip.c                                 ~ tip.h

  > size_t size() (deraadt@)

  ~ value.c                               

  > more delinting. (moritz@)

== usr.sbin ========================================================== 08/08 ==

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

authpf

  ~ authpf.c                              

  > FILE * leak (deraadt@)

config

  ~ config.8                              

  > Add BUGS section documenting included files should start with a
  > newline or comment. Problem found by mpech@; ok miod@ (otto@)

pkg_add

  ~ OpenBSD/PackingElement.pm             ~ OpenBSD/PackingOld.pm

  > un-special case old constructs: make them inherit from the same class,
  > and remove any special code from Factory.
  > As it's simpler remove local variables. (espie@)

rdconfig

  ~ rdconfig.c                            

  > Nuke unused variable.
  > ok ray@ (dhill@)

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



More information about the odc mailing list