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

fu-smbios: Fix parsing smbios data #4853

Merged
merged 1 commit into from Jul 20, 2022
Merged

fu-smbios: Fix parsing smbios data #4853

merged 1 commit into from Jul 20, 2022

Conversation

superm1
Copy link
Member

@superm1 superm1 commented Jul 20, 2022

When support was added for falling back to SMBIOS data from the kernel
in /sys/class/dmi, we inadvertently stopped caring about the data parsed
directly from DMI tables as first priority. This caused a regression in
hwids from some OEMs that relied upon IDs that could only be properly built
from DMI tables, not the kernel /sys/class/dmi interface.

Link: https://bugs.launchpad.net/ubuntu/+source/fwupd/+bug/1982103
Fixes: 464425f ("SMBIOS: try reading from /sys/class/dmi if direct access fails")
Signed-off-by: Mario Limonciello mario.limonciello@amd.com

Type of pull request:

@superm1 superm1 requested review from hughsie and tari July 20, 2022 21:07
libfwupdplugin/fu-smbios.c Outdated Show resolved Hide resolved
When support was added for falling back to SMBIOS data from the kernel
in /sys/class/dmi, we inadvertently stopped caring about the data parsed
directly from DMI tables as first priority.  This caused a regression in
hwids from some OEMs that relied upon IDs that could only be properly built
from DMI tables, not the kernel /sys/class/dmi interface.

Link: https://bugs.launchpad.net/ubuntu/+source/fwupd/+bug/1982103
Fixes: 464425f ("SMBIOS: try reading from /sys/class/dmi if direct access fails")
Signed-off-by: Mario Limonciello <mario.limonciello@amd.com>
Copy link
Member

@hughsie hughsie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for hunting this one down.

@superm1 superm1 enabled auto-merge (rebase) July 20, 2022 21:15
@superm1
Copy link
Member Author

superm1 commented Jul 20, 2022

Thanks for hunting this one down.

Sure. It's kinda a bad one. Between that and the one with the devone (and anything based off Insyde really) being broken, care for a new fixup release?

@superm1 superm1 merged commit 445a8af into main Jul 20, 2022
@superm1 superm1 deleted the mlimonci/fix-smbios branch July 20, 2022 21:23
@hughsie
Copy link
Member

hughsie commented Jul 20, 2022

care for a new fixup release

Planned for Monday.

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

Successfully merging this pull request may close these issues.

None yet

2 participants