Audit return codes (and handle failure correctly) for: * request_region() * request_irq() * kmalloc(), vmalloc(), etc * register_netdev() has to be checked as well * misc_register() (yes, it can fail, murphy's law applies here as well) * scsi_register() * create_proc_*() * pci_map_* might return 0 for a valid mapping. * Some code tests mapping for a non-zero value, which is incorrect. * get_free_pages() and get_free_page() * ioremap() -- Some are using this as a pointer, which is wrong. * put/get_user() * copy_*_user(), unlike put_user() it returns number of bytes failed to copy. * kernel_thread() * IDE drivers, as it can cause a real problem: * [D: http://marc.theaimsgroup.com/?l=kernel-janitor-discuss&m=110416923101031]