New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fix(linux): validate cpu fields length before accessing index #1544

Merged

shirou merged 2 commits into shirou:master from JanDeDobbeleer:master

Oct 30, 2023

Contributor

JanDeDobbeleer commented Oct 30, 2023

Two fixes in this one:

validate the fields length before assuming we have everything. This is related to Linux i686 JanDeDobbeleer/oh-my-posh#4397
do not print things to stdout on Windows. This is related to gopsutil: unexpected processor queue length counter error JanDeDobbeleer/oh-my-posh#2947

JanDeDobbeleer added 2 commits

October 30, 2023 08:35


          fix(linux): validate cpu fields length before accessing index

a1eedca


          fix(windows): remove log statement

ac63a56

github-actions bot added package:cpu package:load labels

shirou approved these changes

View reviewed changes

Owner

shirou left a comment

Great patch. Thank you for your contribution!

shirou merged commit a7e2bdf into shirou:master

19 checks passed

tinyluck mentioned this pull request

linux386版本npc在iOS的iSH虚拟机里运行出现index out of range错误 yisier/nps#146

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package:cpu package:load