[odc] Daily src changes for 2008-01-24

ODC auto at squish.net
Fri Jan 25 07:00:02 GMT 2008


OpenBSD src changes summary for 2008-01-24
==========================================

regress/usr.bin                         sbin/disklabel
share/man                               sys/arch/amd64/stand/installboot
sys/arch/i386/stand/installboot         sys/arch/sparc64/conf
sys/arch/sparc64/dev                    sys/dev
sys/dev/usb                             usr.bin/pcc
usr.sbin/httpd                          usr.sbin/snmpd

== regress =========================================================== 01/06 ==

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

usr.bin

  ~ pcc/ccom/Makefile                     + pcc/ccom/mustfail0000.c
  + pcc/ccom/mustpass0000.c               + pcc/ccom/mustpass0001.c

  > Pull testcases from master repo. (stefan@)

== sbin ============================================================== 02/06 ==

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

disklabel

  ~ disklabel.c                           

  > "read(..., ..., sizeof Y) < sizeof Y" is a dangerous idiom because it
  > does an unsigned comparison and read() can return -1. Use '!=' instead
  > of '<' since read() can't return more than 'sizeof Y'. Not perfect
  > (that would require a separate test for -1) but a very common usage.
  > Do the same for a write(), and a couple of read() calls which are
  > probably ok but why be inconsistant?
  > This, as the last couple of commits, found by ian@ as a result of
  > poking around in fsck_msdos() when his new iPod had problems vs
  > OpenBSD.
  > Concept reviewed by miod@, beck@, otto@ and ian at . (krw@)

== share ============================================================= 03/06 ==

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

man

  ~ man4/man4.sparc64/beeper.4            

  > Finish the beep(4) driver by hooking it up to the ukbd(4) beep handler
  > and disable the one second long beep on attach.
  > ok miod@ (robert@)

  ~ man4/softraid.4                       

  > Add RAID 0 and clean up some of the text. (marco@)

  ~ man4/inet6.4                          

  > -r1.23 introduced an error in signposting, as spotted by Stefan Sperling;
  > fix that now;
  > while i'm here, the text read very badly, so i've redone the paragraph;
  > (jmc@)

== sys =============================================================== 04/06 ==

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

arch/amd64/stand/installboot

  ~ installboot.c                         

  > "read(..., ..., sizeof Y) < sizeof Y" is a dangerous idiom because it
  > does an unsigned comparison and read() can return -1. Use '!=' instead
  > of '<' since read() can't return more than 'sizeof Y'. Not perfect
  > (that would require a separate test for -1) but a very common usage.
  > ok toby@ (krw@)

arch/i386/stand/installboot

  ~ installboot.c                         

  > "read(..., ..., sizeof Y) < sizeof Y" is a dangerous idiom because it
  > does an unsigned comparison and read() can return -1. Use '!=' instead
  > of '<' since read() can't return more than 'sizeof Y'. Not perfect
  > (that would require a separate test for -1) but a very common usage.
  > ok toby@ (krw@)

arch/sparc64/conf

  ~ GENERIC                               

  > Finish the beep(4) driver by hooking it up to the ukbd(4) beep handler
  > and disable the one second long beep on attach.
  > ok miod@ (robert@)

arch/sparc64/dev

  ~ beep.c                                

  > Finish the beep(4) driver by hooking it up to the ukbd(4) beep handler
  > and disable the one second long beep on attach.
  > ok miod@ (robert@)

dev

  ~ softraid.c                            

  > KNF (marco@)

  ~ softraid_raid0.c                      

  > Oops, make fit in 80 cols. (marco@)

  ~ softraidvar.h                         ~ softraid_raid0.c

  > Add IO path to RAID 0 discipline. (marco@)

  ~ softraid.c                            ~ softraid_crypto.c
  ~ softraid_raid0.c                      ~ softraid_raid1.c

  > Bring softraid into the world of 16 byte commands; this allows for > 2TB
  > disks
  > Fix bioctl size output which was off by *512; diagnosed by otto (marco@)

  ~ softraid.c                            ~ softraid_raid0.c
  ~ softraid_raid1.c                      ~ softraidvar.h

  > Create chunk and state transition functions for RAID 0.
  > Move RAID 1 chunk and state transition functions into proper file.
  > Let Crypto use RAID 1 chunk and state transition functions for now but this
  > needs fixing. (marco@)

dev/usb

  ~ ukbd.c                                ~ ukbdvar.h

  > provide an external hook for beeper devices just like pckbd(4) does
  > ok miod@ (robert@)

  ~ if_upgt.c                             

  > Report RX rates in radio tap. (mglocker@)

  ~ if_upgt.c                             ~ if_upgtvar.h

  > We don't report the antenna in the bpf tap.  Remove it. (mglocker@)

  ~ if_upgt.c                             ~ if_upgtvar.h

  > Fix some comments. (mglocker@)

== usr.bin =========================================================== 05/06 ==

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

pcc

  + sparc64/code.c                        + sparc64/local.c
  + sparc64/local2.c                      + sparc64/macdefs.h
  + sparc64/order.c                       + sparc64/table.c

  > Work-in-progress for sparc64, by David Crawshaw. (ragge@)

== usr.sbin ========================================================== 06/06 ==

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

httpd

  ~ src/main/http_protocol.c              

  > "read(..., ..., sizeof Y) < sizeof Y" is a dangerous idiom because it
  > does an unsigned comparison and read() can return -1. Use '!=' instead
  > of '<' since read() can't return more than 'sizeof Y'. Not perfect
  > (that would require a separate test for -1) but a very common usage.
  > ok henning@ (krw@)

snmpd

  ~ trap.c                                

  > timeticks type (reyk@)

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


More information about the odc mailing list