]> git.proxmox.com Git - mirror_ubuntu-eoan-kernel.git/commitdiff
platform/x86: surface3-wmi: Balance locking on error path
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 15 Dec 2016 01:10:02 +0000 (03:10 +0200)
committerDarren Hart <dvhart@linux.intel.com>
Sun, 18 Dec 2016 22:56:43 +0000 (14:56 -0800)
There is a possibility that lock will be left acquired.
Consolidate error path under out_free_unlock label.

Reported-by: kbuild test robot <fengguang.wu@intel.com>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
drivers/platform/x86/surface3-wmi.c

index 5553b2b85e0a2181dfe3b97930dd2a5e3e9eb250..cbf4d83a727106ee0f7e42ca1b868616d994c0c3 100644 (file)
@@ -60,10 +60,10 @@ static DEFINE_MUTEX(s3_wmi_lock);
 
 static int s3_wmi_query_block(const char *guid, int instance, int *ret)
 {
+       struct acpi_buffer output = { ACPI_ALLOCATE_BUFFER, NULL };
        acpi_status status;
        union acpi_object *obj;
-
-       struct acpi_buffer output = { ACPI_ALLOCATE_BUFFER, NULL };
+       int error = 0;
 
        mutex_lock(&s3_wmi_lock);
        status = wmi_query_block(guid, instance, &output);
@@ -77,13 +77,14 @@ static int s3_wmi_query_block(const char *guid, int instance, int *ret)
                               obj->type == ACPI_TYPE_BUFFER ?
                                                obj->buffer.length : 0);
                }
-               kfree(obj);
-               return -EINVAL;
+               error = -EINVAL;
+               goto out_free_unlock;
        }
        *ret = obj->integer.value;
+ out_free_unlock:
        kfree(obj);
        mutex_unlock(&s3_wmi_lock);
-       return 0;
+       return error;
 }
 
 static inline int s3_wmi_query_lid(int *ret)