On Tue, 09 Feb 2010 15:29:22 -0500 Nicolas Pitre <nico@xxxxxxxxxxx> wrote: > + BUG_ON(!func); > + BUG_ON(!func->card); > + > + host = func->card->host; It's a bit redundant to check for null and then deref the pointer - the kernel will reliably oops on the NULL deref, which gives us the same info. I guess the BUG_ON is more useful in the case where a double-deref is being performed, as it can otherwise be a bit hard to work out which pointer was NULL. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
