Commit 0a91e551 authored by Colin Ian King's avatar Colin Ian King Committed by Len Brown

tools/power turbostat: fix error case overflow read of slm_freq_table[]

When i >= SLM_BCLK_FREQS, the frequency read from the slm_freq_table
is off the end of the array because msr is set to 3 rather than the
actual array index i.  Set i to 3 rather than msr to fix this.
Signed-off-by: default avatarColin Ian King <colin.king@canonical.com>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent 01a67adf
...@@ -3036,7 +3036,7 @@ double slm_bclk(void) ...@@ -3036,7 +3036,7 @@ double slm_bclk(void)
i = msr & 0xf; i = msr & 0xf;
if (i >= SLM_BCLK_FREQS) { if (i >= SLM_BCLK_FREQS) {
fprintf(outf, "SLM BCLK[%d] invalid\n", i); fprintf(outf, "SLM BCLK[%d] invalid\n", i);
msr = 3; i = 3;
} }
freq = slm_freq_table[i]; freq = slm_freq_table[i];
......
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