On Fri, Jul 27, 2007 at 10:04:51AM -0400, Jeff Garzik wrote: > Regardless of the original code, it is quite irregular to force the > caller to clean up after advansys_wide_init_chip() kmalloc failure. I > could understand it, perhaps, if err_free_wide_mem label was used more > than once, but it's not. It is ... in the next patch. -- "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step." - To unsubscribe from this list: 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