summaryrefslogtreecommitdiff
path: root/drivers/acpi/pci_link.c
diff options
context:
space:
mode:
authorHenrique de Moraes Holschuh <hmh@hmh.eng.br>2008-06-04 02:36:11 (GMT)
committerLen Brown <len.brown@intel.com>2008-06-11 23:13:45 (GMT)
commit24e45bbe695719dca8c20e03d386eb6ea86526b5 (patch)
treedb4b154bbeffe64d275bdfc05a0cc32259e1e54f /drivers/acpi/pci_link.c
parent9c0a76e16ee6648f4bd19563e9fe12a4f4fabba1 (diff)
downloadlinux-fsl-qoriq-24e45bbe695719dca8c20e03d386eb6ea86526b5.tar.xz
thinkpad-acpi: fix LED handling on older ThinkPads
The less tested codepaths for LED handling, used on ThinkPads 570, 600e/x, 770e, 770x, A21e, A2xm/p, T20-22, X20 and maybe a few others, would write data to kernel memory it had no business touching, for leds number 3 and above. If one is lucky, that illegal write would cause an OOPS, but chances are it would silently corrupt a byte. The problem was introduced in commit af116101, "ACPI: thinkpad-acpi: add sysfs led class support to thinkpad leds (v3.2)". Fix the bug by refactoring the entire code to be far more obvious on what it wants to do. Also do some defensive "constification". Issue reported by Karol Lewandowski <lmctlx@gmail.com> (he's an lucky guy and got an OOPS instead of silent corruption :-) ). Root cause of the OOPS identified by Adrian Bunk <bunk@kernel.org>. Thanks, Adrian! Signed-off-by: Henrique de Moraes Holschuh <hmh@hmh.eng.br> Tested-by: Karol Lewandowski <lmctlx@gmail.com> Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi/pci_link.c')
0 files changed, 0 insertions, 0 deletions