The variable was dereferenced only if it was NULL (sic)... Spotted by the Coverity checker. Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx> --- linux-2.6.16-rc5-mm3-full/drivers/scsi/ibmmca.c.old 2006-03-10 20:39:03.000000000 +0100 +++ linux-2.6.16-rc5-mm3-full/drivers/scsi/ibmmca.c 2006-03-10 20:41:47.000000000 +0100 @@ -2357,8 +2357,7 @@ static int ibmmca_proc_info(struct Scsi_ spin_lock_irqsave(hosts[i]->host_lock, flags); /* Check it */ host_index = i; if (!shpnt) { - len += sprintf(buffer + len, "\nIBM MCA SCSI: Can't find adapter for host number %d\n", - shpnt->host_no); + len += sprintf(buffer + len, "\nIBM MCA SCSI: Can't find adapter"); return len; } max_pun = subsystem_maxid(host_index); - : 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