On Wed, 2005-11-30 at 11:43 -0800, Mark Haverkamp wrote: > On Wed, 2005-11-30 at 19:14 +0000, Christoph Hellwig wrote: > > On Wed, Nov 30, 2005 at 10:36:42AM -0800, Mark Haverkamp wrote: > > > Received from Mark Salyzyn. > > > > > > scsi_bios_ptable return value is not being checked in aac_biosparm. > > > > should probably go into 2.6.15 after the usual trivial codingstyle fix.. > > > > > buf = scsi_bios_ptable(bdev); > > > + if(buf == NULL) > > > + return 0; > > > > if (!buf) > > return 0; > OK, here is the updated patch: I think your space bar is defective on your keyboard > buf = scsi_bios_ptable(bdev); > + if(!buf) > + return 0; in linux we put a space between the if and the ()'s - : send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
