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

[ODC]

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

gnu/usr.sbin/sendmail                   sbin/disklabel
sbin/pfctl                              sys/arch/i386/i386
sys/dev/pci                             sys/dev/usb
sys/net                                 usr.sbin/httpd
usr.sbin/inetd                          

== gnu =============================================================== 01/04 ==

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

usr.sbin/sendmail

  ~ sendmail/conf.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.
  > We don't actually compile this code so there is no functional change.
  > Diff fixed & ok millert@ (krw@)

== sbin ============================================================== 02/04 ==

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

disklabel

  ~ disklabel.8                           

  > document maximum partition size; ok jmc@ krw@ (otto@)

  ~ editor.c                              

  > Fix my screwup and don't return NULL from sort_paritions.
  > Problem found by ckuethe@, slightly different fix than the one
  > proposed by otto@ applied. (krw@)

pfctl

  ~ pfctl_optimize.c                      

  > Get rid of warning when compiling with OPT_DEBUG. (mcbride@)

== sys =============================================================== 03/04 ==

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

arch/i386/i386

  ~ machdep.c                             

  > Add support for some other CPU's.  Thank you to
  > jahrens at centtech for the information.
  > ok otto@, hshoexer@, beck@ (weingart@)

dev/pci

  ~ pcidevs.h                             ~ pcidevs_data.h

  > regen (brad@)

  ~ pcidevs                               

  > - correct a typo for the Atheros AR5311 entry.
  > - add a few Atheros AR5212 compatible PCI ids. From the Linux ath5k driver.
  > ok reyk@ dlg@ (brad@)

dev/usb

  ~ if_upgt.c                             

  > Since TX rate control is done by the firmware, we report the maximum
  > rate which is available. (mglocker@)

net

  ~ bpf.c                                 

  > Prevent USB network devices to generate a page fault trap when detached
  > while UP and holding an open bpf handler by checking bpfilter_lookup()
  > for returning NULL in bpfpoll().  Added an XXX comment which reminds us
  > to recheck why this race condition happens in conjunction with the USB
  > stack.
  > Commented by miod@ and thib@ (would prefer to directly fix race condition,
  > if this is possible at all).
  > lot of help and OK claudio@ (mglocker@)

== usr.sbin ========================================================== 04/04 ==

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

httpd

  ~ conf/httpd.conf                       

  > show an example for writing ErrorLog to syslog
  > ok henning (sthen@)

inetd

  ~ inetd.c                               

  > validate argv, okay otto@, henning@ (espie@)

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