wmi: check find_guid() return value to prevent oops
authorPaul Rolland <rol@as2917.net>
Wed, 30 Dec 2009 06:07:40 +0000 (01:07 -0500)
committerLen Brown <len.brown@intel.com>
Wed, 30 Dec 2009 07:48:55 +0000 (02:48 -0500)
Signed-off-by: Paul Rolland <rol@as2917.net>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/platform/x86/wmi.c

index 9f93d6c0f510d88a9e662caa917e92df4a4ed39f..cc9ad740bda15266303312fe68ab2586cc1bcfd3 100644 (file)
@@ -492,8 +492,7 @@ wmi_notify_handler handler, void *data)
        if (!guid || !handler)
                return AE_BAD_PARAMETER;
 
-       find_guid(guid, &block);
-       if (!block)
+       if (!find_guid(guid, &block))
                return AE_NOT_EXIST;
 
        if (block->handler)
@@ -521,8 +520,7 @@ acpi_status wmi_remove_notify_handler(const char *guid)
        if (!guid)
                return AE_BAD_PARAMETER;
 
-       find_guid(guid, &block);
-       if (!block)
+       if (!find_guid(guid, &block))
                return AE_NOT_EXIST;
 
        if (!block->handler)