From error27@gmail.com Thu Jan 14 16:25:22 2010 From: Dan Carpenter Date: Mon, 28 Dec 2009 18:59:01 +0200 Subject: Staging: comedi: reorder check in pcmmio_attach() To: Greg Kroah-Hartman Cc: devel@driverdev.osuosl.org, Ian Abbott , Frank Mori Hess Message-ID: <20091228165901.GC17645@bicker> Content-Disposition: inline The check for dio_num_asics is used to determine if there is more than 1 irq. If it is false then irq[1] is past the end of the array. Signed-off-by: Dan Carpenter Signed-off-by: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/pcmmio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/staging/comedi/drivers/pcmmio.c +++ b/drivers/staging/comedi/drivers/pcmmio.c @@ -550,7 +550,7 @@ static int pcmmio_attach(struct comedi_d if (irq[0]) { printk("irq: %u ", irq[0]); - if (irq[1] && thisboard->dio_num_asics == 2) + if (thisboard->dio_num_asics == 2 && irq[1]) printk("second ASIC irq: %u ", irq[1]); } else { printk("(IRQ mode disabled) ");