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

Fix uefi parse of ASUS UEFI CAP files #128

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

0lvin
Copy link
Contributor

@0lvin 0lvin commented Aug 13, 2024

Checked with: https://dlcdnets.asus.com/pub/ASUS/mb/BIOS/Pro-WS-WRX90E-SAGE-SE-ASUS-0502.zip

Workaround for:

  File "uefi_firmware/guids/__init__.py", line 21, in get_guid_name
    raw_guid = aguid(guid)
               ^^^^^^^^^^^
  File "/home/denis/Develop/linux-kernel/checks/uefi-firmware-parser/uefi_firmware/utils.py", line 105, in aguid
    a, b, c, d = struct.unpack("%sIHH8s" % (">" if big else "<"), b)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
struct.error: unpack requires a buffer of 16 bytes

Checked with: https://dlcdnets.asus.com/pub/ASUS/mb/BIOS/Pro-WS-WRX90E-SAGE-SE-ASUS-0502.zip

Workaround for:
  File "uefi_firmware/guids/__init__.py", line 21, in get_guid_name
    raw_guid = aguid(guid)
               ^^^^^^^^^^^
  File "/home/denis/Develop/linux-kernel/checks/uefi-firmware-parser/uefi_firmware/utils.py", line 105, in aguid
    a, b, c, d = struct.unpack("%sIHH8s" % (">" if big else "<"), b)
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
struct.error: unpack requires a buffer of 16 bytes
@0lvin 0lvin changed the title Fix uefi parse of ASUS bios CAP files Fix uefi parse of ASUS UEFI CAP files Aug 13, 2024
@theopolis
Copy link
Owner

Thank you! I will check in on the MacOS builders, fix those, then get this merged ASAP.

@theopolis theopolis closed this Oct 1, 2024
@theopolis theopolis reopened this Oct 1, 2024
@theopolis theopolis merged commit 0c5fb74 into theopolis:master Oct 1, 2024
13 of 14 checks passed
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