[PATCH] Don't check pointer for NULL before passing it to kfree [arch/powerpc/kernel...
authorJesper Juhl <jesper.juhl@gmail.com>
Sat, 4 Feb 2006 19:35:59 +0000 (20:35 +0100)
committerPaul Mackerras <paulus@samba.org>
Tue, 7 Feb 2006 10:51:53 +0000 (21:51 +1100)
Checking a pointer for NULL before passing it to kfree is pointless, kfree
does its own NULL checking of input.

Signed-off-by: Jesper Juhl <jesper.juhl@gmail.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
arch/powerpc/kernel/rtas_flash.c

index 5050009..aaf384c 100644 (file)
@@ -672,8 +672,7 @@ static void rtas_flash_firmware(int reboot_type)
 static void remove_flash_pde(struct proc_dir_entry *dp)
 {
        if (dp) {
-               if (dp->data != NULL)
-                       kfree(dp->data);
+               kfree(dp->data);
                dp->owner = NULL;
                remove_proc_entry(dp->name, dp->parent);
        }