Commit 9b5ac668 authored by Naoya Horiguchi's avatar Naoya Horiguchi Committed by Kleber Sacilotto de Souza

tools/power turbostat: fix buffer overrun

BugLink: https://bugs.launchpad.net/bugs/1845405

[ Upstream commit eeb71c95 ]

turbostat could be terminated by general protection fault on some latest
hardwares which (for example) support 9 levels of C-states and show 18
"tADDED" lines. That bloats the total output and finally causes buffer
overrun.  So let's extend the buffer to avoid this.
Signed-off-by: default avatarNaoya Horiguchi <n-horiguchi@ah.jp.nec.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
Signed-off-by: default avatarConnor Kuehl <connor.kuehl@canonical.com>
Signed-off-by: default avatarKleber Sacilotto de Souza <kleber.souza@canonical.com>
parent 697f466a
...@@ -3566,7 +3566,7 @@ int initialize_counters(int cpu_id) ...@@ -3566,7 +3566,7 @@ int initialize_counters(int cpu_id)
void allocate_output_buffer() void allocate_output_buffer()
{ {
output_buffer = calloc(1, (1 + topo.num_cpus) * 1024); output_buffer = calloc(1, (1 + topo.num_cpus) * 2048);
outp = output_buffer; outp = output_buffer;
if (outp == NULL) if (outp == NULL)
err(-1, "calloc output buffer"); err(-1, "calloc output buffer");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment