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

ODC auto at squish.net
Thu Mar 16 07:00:38 GMT 2006


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

bin/md5                                 distrib/miniroot
distrib/zaurus                          etc/etc.cats/MAKEDEV
etc/etc.cats/MAKEDEV.md                 etc/etc.macppc/MAKEDEV
etc/etc.macppc/MAKEDEV.md               etc/etc.mvmeppc/MAKEDEV.md
etc/etc.zaurus/MAKEDEV                  etc/etc.zaurus/MAKEDEV.md
etc/etc.zaurus/sysctl.conf              games/snake
lib/libc                                regress/lib
regress/usr.bin                         share/man
sys/arch/alpha/alpha                    sys/arch/amd64/amd64
sys/arch/amd64/include                  sys/arch/arm/arm
sys/arch/hp300/dev                      sys/arch/hp300/hp300
sys/arch/hppa/dev                       sys/arch/hppa/hppa
sys/arch/hppa64/hppa64                  sys/arch/i386/i386
sys/arch/i386/include                   sys/arch/i386/isa
sys/arch/luna88k/luna88k                sys/arch/mac68k/dev
sys/arch/mac68k/mac68k                  sys/arch/macppc/macppc
sys/arch/macppc/pci                     sys/arch/mips64/mips64
sys/arch/mvme68k/dev                    sys/arch/mvme68k/mvme68k
sys/arch/mvme88k/dev                    sys/arch/mvme88k/mvme88k
sys/arch/mvmeppc/mvmeppc                sys/arch/solbourne/solbourne
sys/arch/sparc/dev                      sys/arch/sparc/include
sys/arch/sparc/sparc                    sys/arch/sparc64/dev
sys/arch/sparc64/sparc64                sys/arch/vax/mscp
sys/arch/vax/vax                        sys/arch/zaurus/conf
sys/ddb                                 sys/dev
sys/dev/ic                              sys/dev/isa
sys/dev/ofw                             sys/dev/pci
sys/dev/wscons                          sys/kern
sys/scsi                                sys/sys
usr.bin/cvs                             usr.bin/logger
usr.bin/rcs                             usr.bin/ssh
usr.bin/vi                              usr.sbin/apm
usr.sbin/apmd                           usr.sbin/bgpd
usr.sbin/ospfd                          usr.sbin/spamdb

== bin =============================================================== 01/10 ==

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

md5

  ~ md5.c                                 

  > Fix signed/unsigned comparisons by using size_t
  > Sprinkle some const
  > ok ray@ jaredy@ (dhill@)

== distrib =========================================================== 02/10 ==

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

miniroot

  ~ install.sub                           

  > Make 'no' the default answer to 'Display the list of XXX servers?'.
  > Suggested by otto@, various people chiming in to agree.
  > While here fix one ask_yn misuse and update copyright to 2006. (krw@)

zaurus

  ~ ramdisk/install.md                    

  > no more need for aperture (deraadt@)

== etc =============================================================== 03/10 ==

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

etc.cats/MAKEDEV

  ~ etc.cats/MAKEDEV                      

  > sync (deraadt@)

etc.cats/MAKEDEV.md

  ~ etc.cats/MAKEDEV.md                   

  > /dev/reload is an old amiga thing, how did it make it here? (deraadt@)

etc.macppc/MAKEDEV

  ~ etc.macppc/MAKEDEV                    

  > sync (deraadt@)

etc.macppc/MAKEDEV.md

  ~ etc.macppc/MAKEDEV.md                 

  > /dev/reload is an old amiga thing, how did it make it here? (deraadt@)

etc.mvmeppc/MAKEDEV.md

  ~ etc.mvmeppc/MAKEDEV.md                

  > /dev/reload is an old amiga thing, how did it make it here? (deraadt@)

etc.zaurus/MAKEDEV

  ~ etc.zaurus/MAKEDEV                    

  > no more need for the aperture (deraadt@)

etc.zaurus/MAKEDEV.md

  ~ etc.zaurus/MAKEDEV.md                 

  > no more need for aperture (deraadt@)

etc.zaurus/sysctl.conf

  ~ etc.zaurus/sysctl.conf                

  > no more need for the aperture (deraadt@)

== games ============================================================= 04/10 ==

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

snake

  ~ snake.c                               

  > Remove useless getuid() check.
  > OK moritz@ jaredy@ (dhill@)

== lib =============================================================== 05/10 ==

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

libc

  ~ stdlib/getopt.3                       

  > Removes extraneous variables from getopt(3).  Inspired by one of
  > David Hill's diffs.
  > ok otto (ray@)

  ~ nls/sv.msg                            

  > spelling, jon at vexed.seeraadt@)

== regress =========================================================== 06/10 ==

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

lib

  ~ libm/Makefile                         

  > add exp & sort (otto@)

  + libm/exp/Makefile                     + libm/exp/exp.c

  > Simple exp(3) and expf(3) regression test for +Inf and -Inf (otto@)

usr.bin

  ~ rcs/Makefile                          ~ rcs/rcsmerge.out

  > Fix and enable rcsmerge test.
  > rcsmerge stuff ok niallo (ray@)

== share ============================================================= 07/10 ==

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

man

  ~ man8/man8.alpha/MAKEDEV.8             ~ man8/man8.amd64/MAKEDEV.8
  ~ man8/man8.cats/MAKEDEV.8              ~ man8/man8.hp300/MAKEDEV.8
  ~ man8/man8.hppa/MAKEDEV.8              ~ man8/man8.hppa64/MAKEDEV.8
  ~ man8/man8.i386/MAKEDEV.8              ~ man8/man8.luna88k/MAKEDEV.8
  ~ man8/man8.mac68k/MAKEDEV.8            ~ man8/man8.macppc/MAKEDEV.8
  ~ man8/man8.mvme68k/MAKEDEV.8           ~ man8/man8.mvme88k/MAKEDEV.8
  ~ man8/man8.sgi/MAKEDEV.8               ~ man8/man8.sparc/MAKEDEV.8
  ~ man8/man8.sparc64/MAKEDEV.8           ~ man8/man8.vax/MAKEDEV.8
  ~ man8/man8.zaurus/MAKEDEV.8            

  > sync (deraadt@)

  ~ man4/xf86.4                           

  > mention machdep.allowaperture can be lowered
  > (and remove mention of XFree86 snuck in here by someone who did NOT
  > understand how wrong that is) (deraadt@)

== sys =============================================================== 08/10 ==

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

arch/alpha/alpha

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

  ~ machdep.c                             

  > use sysctl_int_lower() for the aperture variable.  This lets root close
  > the aperture without having to reboot, but does not allow re-opening;
  > ok matthieu (deraadt@)

arch/amd64/amd64

  ~ powernow-k8.c                         

  > knf (deraadt@)

  ~ autoconf.c                            

  > Correct checks against cd_ndevs - valid numbers are strictly inferior to
  > cd_ndevs. (miod@)

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

  ~ machdep.c                             

  > use sysctl_int_lower() for the aperture variable.  This lets root close
  > the aperture without having to reboot, but does not allow re-opening;
  > ok matthieu (deraadt@)

arch/amd64/include

  ~ db_machdep.h                          

  > remove dead macros (mickey@)

  ~ pmap.h                                

  > Remove address range KASSERT check in vtopte() and kvtopte(), callers are
  > supposed to know which one of the two is applicable when the address is not
  > guaranteed to be a kernel address.
  > ok kettenis@ mickey@ (miod@)

arch/arm/arm

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

  ~ arm32_machdep.c                       

  > use sysctl_int_lower() for the aperture variable.  This lets root close
  > the aperture without having to reboot, but does not allow re-opening;
  > ok matthieu (deraadt@)

arch/hp300/dev

  ~ hd.c                                  

  > Nuke dk_establish(), no longer used. (miod@)

arch/hp300/hp300

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/hppa/dev

  ~ sti_sgc.c                             

  > Correct checks against cd_ndevs - valid numbers are strictly inferior to
  > cd_ndevs. (miod@)

arch/hppa/hppa

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/hppa64/hppa64

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/i386/i386

  ~ powernow-k7.c                         ~ powernow-k8.c

  > spacing (deraadt@)

  ~ autoconf.c                            

  > Correct checks against cd_ndevs - valid numbers are strictly inferior to
  > cd_ndevs. (miod@)

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

  ~ machdep.c                             

  > use sysctl_int_lower() for the aperture variable.  This lets root close
  > the aperture without having to reboot, but does not allow re-opening;
  > ok matthieu (deraadt@)

arch/i386/include

  ~ db_machdep.h                          

  > remove dead macros (mickey@)

  ~ vmparam.h                             

  > allow for more physmem segments; as required by some dell from dhill@
  > (mickey@)

arch/i386/isa

  ~ joy.c                                 

  > Entries in cd_devs[] may be NULL, so be sure to check for them in your
  > device open() function. (miod@)

arch/luna88k/luna88k

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/mac68k/dev

  ~ asc.c                                 

  > Entries in cd_devs[] may be NULL, so be sure to check for them in your
  > device open() function. (miod@)

arch/mac68k/mac68k

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/macppc/macppc

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

  ~ machdep.c                             

  > use sysctl_int_lower() for the aperture variable.  This lets root close
  > the aperture without having to reboot, but does not allow re-opening;
  > ok matthieu (deraadt@)

arch/macppc/pci

  ~ vgafb_pci.c                           

  > Remove dead code (SUPPORTS_NON_CONSOLE and MD_DISPLAY_ISA_IOT)
  > ok miod@, drahn@, dim at .atthieu@)

arch/mips64/mips64

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/mvme68k/dev

  ~ zs.c                                  

  > Correct checks against cd_ndevs - valid numbers are strictly inferior to
  > cd_ndevs. (miod@)

  ~ sshdma.c                              ~ vsdma.c
  ~ wdsc.c                                

  > Nuke dk_establish(), no longer used. (miod@)

arch/mvme68k/mvme68k

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/mvme88k/dev

  ~ vs.c                                  

  > Nuke dk_establish(), no longer used. (miod@)

arch/mvme88k/mvme88k

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/mvmeppc/mvmeppc

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/solbourne/solbourne

  ~ autoconf.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/sparc/dev

  ~ flash.c                               ~ scf.c

  > Entries in cd_devs[] may be NULL, so be sure to check for them in your
  > device open() function. (miod@)

  ~ fd.c                                  ~ xd.c
  ~ xy.c                                  

  > Nuke dk_establish(), no longer used. (miod@)

arch/sparc/include

  ~ _types.h                              

  > Switch sparc to device_register() to find out its boot device (when it's a
  > disk). (miod@)

arch/sparc/sparc

  ~ autoconf.c                            ~ disksubr.c

  > Switch sparc to device_register() to find out its boot device (when it's a
  > disk). (miod@)

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/sparc64/dev

  ~ fd.c                                  

  > Nuke dk_establish(), no longer used. (miod@)

arch/sparc64/sparc64

  ~ autoconf.c                            ~ disksubr.c

  > Nuke dk_establish(), no longer used. (miod@)

  ~ machdep.c                             

  > use sysctl_int_lower() for the aperture variable.  This lets root close
  > the aperture without having to reboot, but does not allow re-opening;
  > ok matthieu (deraadt@)

arch/vax/mscp

  ~ mscp_disk.c                           ~ mscp_tape.c

  > Correct checks against cd_ndevs - valid numbers are strictly inferior to
  > cd_ndevs. (miod@)

arch/vax/vax

  ~ disksubr.c                            

  > Nuke dk_establish(), no longer used. (miod@)

arch/zaurus/conf

  ~ GENERIC                               

  > no more need for aperture (deraadt@)

ddb

  ~ db_usrreq.c                           

  > use sysctl_int_lower (old code had the negative bug) (deraadt@)

  ~ db_usrreq.c                           

  > Still allow ddb.console and ddb.panic to be raised if securelevel <= 0;
  > ok deraadt@ (miod@)

dev

  ~ vnd.c                                 

  > Nuke dk_establish(), no longer used. (miod@)

dev/ic

  ~ ami.c                                 

  > sc->sc_dev.dv_xname converted to DEVNAME(sc). makes things a little easier
  > for me to read. (dlg@)

  ~ ami.c                                 

  > split the code in ami_cmd that does the polling commands out into a
  > function of its own. switch callers of ami_cmd that want polled commands
  > over to using ami_poll. (dlg@)

  ~ ami.c                                 

  > clean up ami_start and move the error handling for a failed command from
  > ami_cmd into ami_start. (dlg@)

  ~ ami.c                                 

  > switch the ioctl paths to using ami_start instead of ami_cmd. now the only
  > things left using ami_cmd are the scsi paths. (dlg@)

  ~ midway.c                              

  > Correct checks against cd_ndevs - valid numbers are strictly inferior to
  > cd_ndevs. (miod@)

dev/isa

  ~ fd.c                                  ~ mcd.c

  > Nuke dk_establish(), no longer used. (miod@)

dev/ofw

  ~ ofdisk.c                              ~ ofnet.c

  > Nuke dk_establish(), no longer used. (miod@)

dev/pci

  ~ wdt.c                                 

  > Entries in cd_devs[] may be NULL, so be sure to check for them in your
  > device open() function. (miod@)

  ~ vga_pci.c                             

  > Remove dead code (SUPPORTS_NON_CONSOLE and MD_DISPLAY_ISA_IOT)
  > ok miod@, drahn@, dim at .atthieu@)

  ~ vga_pci.c                             

  > bits of my tree commited by Matthieu by mistake. (miod@)

dev/wscons

  ~ wsdisplay.c                           

  > Correct checks against cd_ndevs - valid numbers are strictly inferior to
  > cd_ndevs. (miod@)

kern

  ~ sys_generic.c                         

  > copyout() the pollfds if poll() gets interrupted. This resets the revents
  > bitmask and makes poll() behave like documented in the man page.
  > OK deraadt@ (claudio@)

  ~ kern_sysctl.c                         

  > add sysctl_int_lower().  This can read, but can only write to a kernel
  > variable if the result is a lowering of the value; tested matthieu
  > (deraadt@)

scsi

  ~ cd.c                                  ~ sd.c

  > Nuke dk_establish(), no longer used. (miod@)

sys

  ~ disk.h                                

  > Nuke dk_establish(), no longer used. (miod@)

  ~ sysctl.h                              

  > add sysctl_int_lower().  This can read, but can only write to a kernel
  > variable if the result is a lowering of the value; tested matthieu
  > (deraadt@)

== usr.bin =========================================================== 09/10 ==

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

cvs

  ~ rcs.c                                 ~ rcsnum.c

  > remove some bizzare idioms; ok ray (deraadt@)

  ~ worklist.h                            

  > - make cvs_worklist members volatile.
  > from deraadt@ (niallo@)

  ~ worklist.c                            

  > - fix typo in comment; an path -> a path (niallo@)

  ~ cvs.c                                 ~ proto.c

  > bogus idiom (deraadt@)

  ~ worklist.h                            

  > no need for one of the volatile (deraadt@)

  ~ buf.c                                 

  > - fchmod file to mode.
  > ok ray@ (niallo@)

  ~ cvs.h                                 ~ release.c
  ~ util.c                                + util.h

  > - add util.h, reorganising a bunch of things and exposing cvs_yesno()
  > function to be used by rcs. (niallo@)

  ~ log.c                                 ~ log.h

  > - remove unused function cvs_putchar()
  > "well then zap it" joris@ (niallo@)

logger

  ~ logger.c                              

  > Both strlen and sizeof return a size_t.  This also fixes a signed/unsigned
  > comparison.
  > ok ray@ moritz@ (dhill@)

rcs

  ~ rcsprog.c                             

  > Don't assume there is an RCS directory just because the filename
  > contains the letters `RCS'.
  > ok niallo (ray@)

  ~ ci.c                                  ~ rcsprog.c
  ~ rcsprog.h                             

  > Move and rename checkin_choose_rcsfile() to rcs_choosefile().
  > ok joris (ray@)

  ~ ci.c                                  ~ rcsclean.c

  > remove some bizzare idioms; ok ray (deraadt@)

  ~ ci.c                                  

  > - check for tty when asking for logmsg or description
  > - add -I[rev]
  > - more verbosity checks
  > OK niallo@, "Works great" ray at .sa@)

  ~ rcsprog.h                             

  > - add util.h, reorganising a bunch of things and exposing cvs_yesno()
  > function to be used by rcs. (niallo@)

  ~ co.c                                  

  > - use shared cvs_yesno() for yesno prompt, fixing an issue spotted by
  > deraadt@
  > - while here, remove unecessary cast also spotted by deraadt@
  > ok ray@ xsa@ (niallo@)

  ~ ci.c                                  

  > ask for log message when it's not specified, got broken in xsa's previous
  > commit.
  > ok niallo@ (joris@)

  ~ rcsdiff.c                             

  > - zap filename arg in rcsdiff_rev() since its not used.
  > ok ray@ (niallo@)

ssh

  ~ ssh-keygen.c                          

  > if no key file are given when printing the DNS host record, use the
  > host key file(s) as default. ok djm@ (jakob@)

vi

  ~ vi/v_txt.c                            

  > Handle ^C correctly, morph it to escape key so the input
  > is correctly finished for a potential replay; if not, simply bail out
  > and notify that something wrong occurs. Callers will cope.
  > Consistent with what vim and Solaris vi do.
  > Fixes a crash described in NetBSD PR11544, fixed by aymeric
  > ok otto ray (pvalchev@)

== usr.sbin ========================================================== 10/10 ==

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

apm

  ~ apm.8                                 ~ apm.c

  > consistently use power *states* and performance *modes*
  > ok beck, mickey (sturm@)

apmd

  ~ apm-proto.h                           ~ apmd.8
  ~ apmd.c                                ~ apmsubr.c

  > consistently use power *states* and performance *modes*
  > ok beck, mickey (sturm@)

bgpd

  ~ rde_attr.c                            

  > Fix a memory leak when communities are set or deleted. This seems to fix
  > Sylvain Coutant memory issues. A lot of patience and testing by Sylvain.
  > (claudio@)

  ~ bgpd.c                                ~ bgpd.h
  ~ config.c                              ~ session.c

  > Allow the control socket to be changed on the command line. Useful if you
  > need to run multiple bgpds on a single box to simulate a IX. This helped
  > me massivly debugging error reports. OK henning@ (claudio@)

  ~ bgpd.8                                

  > Document new -s path option to select an alternate path for the main
  > control
  > socket. (claudio@)

  ~ bgpd.c                                

  > Sync usage with man page (sort arguments). (claudio@)

  ~ rde_rib.c                             

  > In nexthop_compare() if the two passed pointers point to the same object
  > the odds are better than good that there is no difference. (claudio@)

ospfd

  ~ interface.c                           ~ ospfd.h
  ~ parse.y                               

  > Looks like auth-type simple was busted since a long time. We bcmp() a
  > c string with a lenght <= 8 with a buffer of size 8. Now in some cases
  > there can be trailing garbage after the c-string and so the bcmp() fails.
  > Found by Sergey Matveychuk who provided a similar patch.
  > OK norby@ (claudio@)

  ~ parse.y                               

  > There is no need to check iface != NULL. These parser rules are only
  > allowed
  > if iface is set and so the check is superfluous. OK norby@ (claudio@)

spamdb

  ~ spamdb.c                              

  > Add missing function declarations
  > Sprinkle some const
  > OK beck@ (dhill@)

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



More information about the odc mailing list