diff options
author | Xi Wang <xi.wang@gmail.com> | 2011-12-29 04:49:06 (GMT) |
---|---|---|
committer | Matthew Garrett <mjg@redhat.com> | 2012-03-12 14:25:51 (GMT) |
commit | e424fb8cc4e6634c10f8159b1ff5618cf7bab9c6 (patch) | |
tree | 1907ba5a04c791b676de548e62627aa50b795bc7 /drivers | |
parent | 461e74377cfcfc2c0d6bbdfa8fc5fbc21b052c2a (diff) | |
download | linux-fsl-qoriq-e424fb8cc4e6634c10f8159b1ff5618cf7bab9c6.tar.xz |
panasonic-laptop: avoid overflow in acpi_pcc_hotkey_add()
num_sifr could go negative since acpi_pcc_get_sqty() returns -EINVAL
on error. Then it could bypass the sanity check (num_sifr > 255).
The subsequent call to kzalloc() would allocate a small buffer, leading
to a memory corruption.
Signed-off-by: Xi Wang <xi.wang@gmail.com>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/platform/x86/panasonic-laptop.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/platform/x86/panasonic-laptop.c b/drivers/platform/x86/panasonic-laptop.c index 05be30e..ffff8b4 100644 --- a/drivers/platform/x86/panasonic-laptop.c +++ b/drivers/platform/x86/panasonic-laptop.c @@ -562,8 +562,8 @@ static int acpi_pcc_hotkey_add(struct acpi_device *device) num_sifr = acpi_pcc_get_sqty(device); - if (num_sifr > 255) { - ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "num_sifr too large")); + if (num_sifr < 0 || num_sifr > 255) { + ACPI_DEBUG_PRINT((ACPI_DB_ERROR, "num_sifr out of range")); return -ENODEV; } |