diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2016-05-05 16:23:04 +0300 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-05-05 15:28:58 +0200 |
| commit | f18ebc211e259d4f591e39e74b2aa2de226c9a1d (patch) | |
| tree | d651da8932ba87dbac83b5a60a50b5dd1388596b /include/acpi | |
| parent | a508d954af748bc1f5f1d9174f167a8650fca923 (diff) | |
| download | lwn-f18ebc211e259d4f591e39e74b2aa2de226c9a1d.tar.gz lwn-f18ebc211e259d4f591e39e74b2aa2de226c9a1d.zip | |
ACPI / sysfs: fix error code in get_status()
The problem with ornamental, do-nothing gotos is that they lead to
"forgot to set the error code" bugs. We should be returning -EINVAL
here but we don't. It leads to an uninitalized variable in
counter_show():
drivers/acpi/sysfs.c:603 counter_show()
error: uninitialized symbol 'status'.
Fixes: 1c8fce27e275 (ACPI: introduce drivers/acpi/sysfs.c)
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'include/acpi')
0 files changed, 0 insertions, 0 deletions