On Tue, Oct 02, 2012 at 06:53:05PM +0000, Ortiz, Lance E wrote: > Thanks for letting me know. I send out an updated patch removing > the redundant check. But I am wondering if we should leave it in > for safety's sake. There is no safety issue here. kfree() will always accept NULL pointers. It's cleaner to leave it out. > Is having a info message like this from smatch a problem? It's not a problem. We just send you the one email is all. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html