Skip to content
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.

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

beta-16 : prevent strange characters from model name #20

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

bordeux
Copy link

@bordeux bordeux commented Jan 21, 2022

Some devices had strange character caused by model name buffer overflow.
image

This PR hotfixing this issue.

@bordeux bordeux changed the title Fix: Prevent strange characters from Model name Beta16: Prevent strange characters from Model name Jan 21, 2022
@bordeux bordeux changed the title Beta16: Prevent strange characters from Model name beta-16 : prevent strange characters from model name Jan 21, 2022
@vitlav
Copy link

vitlav commented Jan 27, 2022

Can you check your issue with https://github.com/vitlav/hddtemp ?
There are already many fixes since this stalled version.

# hddtemp
/dev/sda: WDC WDS100T2B0A-00SM50: 43°C
/dev/sdb: KINGSTON SEDC500M960G: 41°C
/dev/sdc: WDC WD1001FALS-00E3A0: 39°C
/dev/sdd: ST2000DM008-2FR102: 40°C
/dev/sde: WDC WD1001FALS-00E3A0: 41°C
/dev/sdf: WDC WD20EZRX-00D8PB0: 40°C
/dev/sdg: WDC WD1002FAEX-00Z3A0: 44°C
/dev/sdh: ST2000DM008-2FR102: 41°C
/dev/sdi: WDC WD10EARS-00Y5B1: 41°C
/dev/sdj: FUJITSU MBA3147RC: 43°C
/dev/sdk: WDC WDS100T1R0A-68A4W0: 39°C
/dev/sdl: WDC WDS100T2B0A-00SM50: 39°C
/dev/sdm: WDC WDS100T1R0A-68A4W0: 38°C
/dev/sdn: WDC WDS100T2B0A-00SM50: 39°C
/dev/sdo: WDC WD1001FALS-00E3A0: 38°C
/dev/sdp: WDC WD3000HLFS-01MZUV0: 35°C
/dev/sdq: WDC WD1003FBYX-01Y7B0: 38°C
/dev/sdr: WDC WD1001FALS-00E3A0: 39°C
/dev/nvme0n1: SPCC M.2 PCIe SSD: 46°C

@bordeux
Copy link
Author

bordeux commented Jan 27, 2022

@vitlav looks like your branch already includes my hotfix.

I suggest keeping this PR open as a helpful notice for individuals, like myself a few days ago, who are searching for a software fix. This way, they can be directed to your comment and branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants