Commit 30c345bd authored by Naoya Horiguchi's avatar Naoya Horiguchi Committed by Greg Kroah-Hartman

tools/power turbostat: fix buffer overrun

[ 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>
parent d485c658
...@@ -4953,7 +4953,7 @@ int initialize_counters(int cpu_id) ...@@ -4953,7 +4953,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