ACPI: drop unnecessary local variable from acpi_system_write_wakeup_device()
The LEN variable is unsigned, therefore checking whether it is less than 0 is useless. Also drop the LEN variable, since the COUNT parameter can be used instead. [rjw: Changed the subject.] Signed-off-by: Cyril Roelandt <tipecaml@gmail.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
Родитель
66fd3835ac
Коммит
05bce79e6d
|
@ -362,16 +362,13 @@ acpi_system_write_wakeup_device(struct file *file,
|
|||
struct list_head *node, *next;
|
||||
char strbuf[5];
|
||||
char str[5] = "";
|
||||
unsigned int len = count;
|
||||
|
||||
if (len > 4)
|
||||
len = 4;
|
||||
if (len < 0)
|
||||
return -EFAULT;
|
||||
if (count > 4)
|
||||
count = 4;
|
||||
|
||||
if (copy_from_user(strbuf, buffer, len))
|
||||
if (copy_from_user(strbuf, buffer, count))
|
||||
return -EFAULT;
|
||||
strbuf[len] = '\0';
|
||||
strbuf[count] = '\0';
|
||||
sscanf(strbuf, "%s", str);
|
||||
|
||||
mutex_lock(&acpi_device_lock);
|
||||
|
|
Загрузка…
Ссылка в новой задаче