cvs commit: src/sys/bus/firewire firewire.c fwdev.c fwdma.c fwmem.c fwohci.c src/sys/bus/pci pci.c src/sys/dev/disk/ata atapi-cd.c atapi-fd.c atapi-tape.c src/sys/dev/disk/isp isp_pci.c src/sys/dev/disk/mpt mpt_pci.c src/sys/dev/disk/nata ata-raid.c ...
Jason Smethers
jason at smethers.net
Mon Jan 7 00:37:30 PST 2008
Sascha Wildner wrote:
Remove bogus checks after kmalloc(M_WAITOK) which never returns NULL.
http://www.dragonflybsd.org/cvsweb/src/sys/dev/raid/twa/twa_cam.c.diff?r1=1.5&r2=1.6&f=u
http://www.dragonflybsd.org/cvsweb/src/sys/dev/sound/pci/ds1.c.diff?r1=1.9&r2=1.10&f=u
You might want to revert these back to assigning the allocated memory to
each's respective pointer, instead of always setting the pointer to NULL.
Jason
More information about the Commits
mailing list