lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20071026115859.GJ5155@kernel.dk> Date: Fri, 26 Oct 2007 13:58:59 +0200 From: Jens Axboe <jens.axboe@...cle.com> To: Anton Blanchard <anton@...ba.org> Cc: linux-kernel@...r.kernel.org Subject: Re: [PATCH] Initialise scatter/gather list in ata_sg_setup On Thu, Oct 25 2007, Anton Blanchard wrote: > > (Sorry the subject was incorrect)... > > After turning on DEBUG_SG I hit a fail: > > kernel BUG at include/linux/scatterlist.h:50! > > ata_qc_issue > ata_scsi_translate > ipr_queuecommand > scsi_dispatch_cmd > scsi_request_fn > elv_insert > blk_execute_rq_nowait > blk_execute_rq > sg_io > scsi_cmd_ioctl > cdrom_ioctl > sr_block_ioctl > blkdev_driver_ioctl > blkdev_ioctl > block_ioctl > do_ioctl > vfs_ioctl > sys_ioctl > sg_ioctl_trans > > It looks like ata_sg_setup is working on an uninitialised sg table. Call > sg_init_table to initialise it before use. > > Signed-off-by: Anton Blanchard <anton@...ba.org> > --- > > Jens this is against your sg branch. This fixes the fail but Id > appreciate a confirmation that it's the right fix :) The patch looks OK, you could also get away with just doing that sg_init_table() on qc init/alloc. -- Jens Axboe - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists