]> git.karo-electronics.de Git - karo-tx-linux.git/commit
ACPI / sysfs: make GPE sysfs attributes only accept correct values
authorLan Tianyu <tianyu.lan@intel.com>
Thu, 12 Sep 2013 07:32:05 +0000 (03:32 -0400)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 25 Sep 2013 15:21:34 +0000 (17:21 +0200)
commitc48b156517348a0130e9e6edbeba95f9b4e50d65
tree95c98e48b360593276d7133d4733b84e0cda381b
parent16a26e85279fd672050ffc3637038366629e8653
ACPI / sysfs: make GPE sysfs attributes only accept correct values

According to the design, GPE sysfs attributes should accept "disable",
"enable", "clear" and integer numbers as params. Current code checks
"disable", "enable" and "clear" first. If the param doesn't match,
pass it to strtoul() as a string representing an integer number and
assign the return value to the given GPE count. It is missing the check
of whether or not the param really represents an integer number and
strtoul() will return 0 if the string is not a number.  This causes any
params except for "enable", "disable", "clear" and a number to make the
GPE count become 0. This patch is to use kstrtoul() to replace strtoul()
and check the return value. If the convertion is successful, use as the
new GPE count. If not, return an error.

Signed-off-by: Lan Tianyu <tianyu.lan@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/sysfs.c