diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2021-04-21 17:19:27 +0200 |
|---|---|---|
| committer | Hans de Goede <hdegoede@redhat.com> | 2021-04-21 21:32:31 +0200 |
| commit | 37bd59d3cef8e90055dc26e25d9aba00f06f5f9f (patch) | |
| tree | e57cfe7e34acf290e638b4657ef40f48a147efb5 /tools/perf/util/evsel.h | |
| parent | platform/x86: intel_pmc_core: add ACPI dependency (diff) | |
| download | linux-37bd59d3cef8e90055dc26e25d9aba00f06f5f9f.tar.xz linux-37bd59d3cef8e90055dc26e25d9aba00f06f5f9f.zip | |
platform/x86: intel_pmc_core: Uninitialized data in pmc_core_lpm_latch_mode_write()
The simple_write_to_buffer() can return success if even a single byte
is copied from user space. In this case it can result in using
uninitalized data if the buf[] array is not fully initialized. Really
we should only succeed if the whole buffer is copied.
Just using copy_from_user() is simpler and more appropriate.
Fixes: 8074a79fad2e ("platform/x86: intel_pmc_core: Add option to set/clear LPM mode")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/YIBCf+G9Ef8wrGJw@mwanda
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Diffstat (limited to 'tools/perf/util/evsel.h')
0 files changed, 0 insertions, 0 deletions