platform/x86: intel-wmi-thunderbolt: Silence error cases
authorMario Limonciello <mario.limonciello@dell.com>
Thu, 14 Sep 2017 06:41:55 +0000 (01:41 -0500)
committerDarren Hart (VMware) <dvhart@infradead.org>
Fri, 22 Sep 2017 22:08:19 +0000 (15:08 -0700)
These were raised by Lukas Wunner as potential DOS attacks against
the system log by passing bad data to sysfs.

Signed-off-by: Mario Limonciello <mario.limonciello@dell.com>
Signed-off-by: Darren Hart (VMware) <dvhart@infradead.org>
drivers/platform/x86/intel-wmi-thunderbolt.c

index 32fb6cc..c2257bd 100644 (file)
@@ -41,13 +41,10 @@ static ssize_t force_power_store(struct device *dev,
        if (mode == 0 || mode == 1) {
                status = wmi_evaluate_method(INTEL_WMI_THUNDERBOLT_GUID, 0, 1,
                                             &input, NULL);
-               if (ACPI_FAILURE(status)) {
-                       pr_err("intel-wmi-thunderbolt: failed setting %s\n",
-                              buf);
+               if (ACPI_FAILURE(status))
                        return -ENODEV;
-               }
        } else {
-               pr_err("intel-wmi-thunderbolt: unsupported mode: %d", mode);
+               return -EINVAL;
        }
        return count;
 }