cvs commit: src/sys/dev/bktr bktr_i2c.c src/sys/dev/ichsmb
ichsmb.c src/sys/dev/iicbus iicsmb.c src/sys/dev/smbus smb.c
smb.h smbconf.c smbconf.h smbus.c smbus.h smbus_if.m
src/sys/pci alpm.c amdpm.c amdsmb.c intpm.c nfsmb.c viapm.c
jhb at freebsd.org
Mon Sep 11 15:27:21 PDT 2006
On Monday 11 September 2006 18:08, John-Mark Gurney wrote:
> John Baldwin wrote this message on Mon, Sep 11, 2006 at 20:52 +0000:
> > - Change the SMB_BREAD ioctl to write out the updated struct smbcmd
> > will contain the actual number of bytes read in the 'count' field. To
> > preserve the previous ABI, the old ioctl value is mapped to
> > which doesn't copy the updated smbcmd back out to userland. I doubt
> > actually used the old BREAD anyway as it was rediculous to do a
> > but not tell the using program how much data was actually read.
> > - Fix buffer overflows in the bread() methods of ichsmb(4), alpm(4),
> > amdpm(4), amdsmb(4), intpm(4), and nfsmb(4).
> The old SMB_BREAD w/o the other change would overflow the buffer w/
> the real amount of data which you had no way to know exactly how much
> was overwritten (at least you knew it'd be no more than 32 bytes due to
> smbus limitations)...
> I'd vote for removing _OLD_BREAD, and adding it back when merged into
> 6.x... 7.0 is as good of a time to change the API...
Well, for now I hope to MFC this stuff as the IPMI stuff I'm working on
depends on this to handle IPMI over SMBus (SSIF). Once it is MFC'd, then we
can remove the old one in 7.
More information about the cvs-all