Commit eeb71c95 authored by Naoya Horiguchi's avatar Naoya Horiguchi Committed by Len Brown

tools/power turbostat: fix buffer overrun

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>
parent 605736c6
...@@ -5131,7 +5131,7 @@ int initialize_counters(int cpu_id) ...@@ -5131,7 +5131,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