summaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorColin Ian King <colin.king@canonical.com>2016-04-25 13:03:15 +0100
committerLen Brown <len.brown@intel.com>2016-12-01 01:33:17 -0500
commit0a91e551527a7b931d36f4822b97e70536bd2bb3 (patch)
tree925a0597f1a2185b8265f91c39745a41c786f506 /tools
parent01a67adfc5d73d24e999d9cf65b5b8a6687187b8 (diff)
downloadlinux-0a91e551527a7b931d36f4822b97e70536bd2bb3.tar.gz
linux-0a91e551527a7b931d36f4822b97e70536bd2bb3.tar.bz2
linux-0a91e551527a7b931d36f4822b97e70536bd2bb3.zip
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: Colin Ian King <colin.king@canonical.com> Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'tools')
-rw-r--r--tools/power/x86/turbostat/turbostat.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/power/x86/turbostat/turbostat.c b/tools/power/x86/turbostat/turbostat.c
index 741fefa2f07e..12a6eea2d6ae 100644
--- a/tools/power/x86/turbostat/turbostat.c
+++ b/tools/power/x86/turbostat/turbostat.c
@@ -3036,7 +3036,7 @@ double slm_bclk(void)
i = msr & 0xf;
if (i >= SLM_BCLK_FREQS) {
fprintf(outf, "SLM BCLK[%d] invalid\n", i);
- msr = 3;
+ i = 3;
}
freq = slm_freq_table[i];