hp-wmi: Fix mixing up of and/or directive
authorThomas Renninger <trenn@suse.de>
Thu, 29 Jul 2010 10:27:59 +0000 (12:27 +0200)
committerMatthew Garrett <mjg@redhat.com>
Tue, 3 Aug 2010 13:49:12 +0000 (09:49 -0400)
This should have been an "and". Additionally checking for !obj
is even better.

Signed-off-by: Thomas Renninger <trenn@suse.de>
CC: linux-acpi@vger.kernel.or
CC: platform-driver-x86@vger.kernel.org
CC: mjg@redhat.com
Signed-off-by: Matthew Garrett <mjg@redhat.com>
drivers/platform/x86/hp-wmi.c

index c5f95d1e0315bd416bea0e127e9ee84ea1b3645b..7e8a136b025b3e289ba44033e0ea0fec85df61eb 100644 (file)
@@ -434,7 +434,9 @@ static void hp_wmi_notify(u32 value, void *context)
 
        obj = (union acpi_object *)response.pointer;
 
-       if (obj || obj->type != ACPI_TYPE_BUFFER) {
+       if (!obj)
+               return;
+       if (obj->type != ACPI_TYPE_BUFFER) {
                printk(KERN_INFO "hp-wmi: Unknown response received %d\n",
                       obj->type);
                kfree(obj);