diff options
| author | Joseph Mulloy <jmulloy@edx.org> | 2018-01-26 10:44:44 -0500 |
|---|---|---|
| committer | Stéphane Lesimple <speed47_github@speed47.net> | 2018-01-26 18:23:18 +0100 |
| commit | 7404929661575506651fac64dec7322cd8682de4 (patch) | |
| tree | bfc7cbee2fce18143903769653ac8be2542bae5e | |
| parent | bf46fd5d9b121db209f9fbba25c49431f2fcc3fe (diff) | |
| download | spectre-meltdown-checker-7404929661575506651fac64dec7322cd8682de4.tar.lz spectre-meltdown-checker-7404929661575506651fac64dec7322cd8682de4.tar.xz spectre-meltdown-checker-7404929661575506651fac64dec7322cd8682de4.zip | |
Fix printing of microcode to use cpuinfo values
The values used should be the ones that come from cpuinfo instead of
the test values. The following line will print the last tuple tested
instead of the actual values of the CPU.
Line 689: _debug "is_ucode_blacklisted: no ($model/$stepping/$ucode)"
| -rwxr-xr-x | spectre-meltdown-checker.sh | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/spectre-meltdown-checker.sh b/spectre-meltdown-checker.sh index a19d4c3..36e2a7b 100755 --- a/spectre-meltdown-checker.sh +++ b/spectre-meltdown-checker.sh @@ -681,12 +681,12 @@ is_ucode_blacklisted() stepping=$(( $(echo $tuple | cut -d, -f2) )) ucode=$(echo $tuple | cut -d, -f3) if [ "$cpu_model" = "$model" ] && [ "$cpu_stepping" = "$stepping" ] && echo "$cpu_ucode" | grep -qi "^$ucode$"; then - _debug "is_ucode_blacklisted: we have a match! ($model/$stepping/$ucode)" - bad_ucode_found="Intel CPU Family 6 Model $model Stepping $stepping with microcode $ucode" + _debug "is_ucode_blacklisted: we have a match! ($cpu_model/$cpu_stepping/$cpu_ucode)" + bad_ucode_found="Intel CPU Family 6 Model $cpu_model Stepping $cpu_stepping with microcode $cpu_ucode" return 0 fi done - _debug "is_ucode_blacklisted: no ($model/$stepping/$ucode)" + _debug "is_ucode_blacklisted: no ($cpu_model/$cpu_stepping/$cpu_ucode)" return 1 } |