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

Fixes str vs byte mismatch, and some other minor changes #2

Open

tandasat wants to merge 3 commits into Cr4sh:master from tandasat:master

tandasat commented Dec 28, 2020

This PR includes changes for followings:

Fix or byte vs str mismatches causing exception like below and some other breaks

[+] SMRAM is at 0x88400000:887fffff
Traceback (most recent call last):
  File "smram_parse_orig.py", line 637, in <module>
    sys.exit(main())
  File "smram_parse_orig.py", line 614, in main
    d.dump_smst()
  File "smram_parse_orig.py", line 264, in dump_smst
    ptr = self.data.find('SMST\0\0\0\0')
TypeError: argument should be integer or bytes-like object, not 'str'

Fix of the os.system error on Windows like this

'UEFIDump" "fw_image_1609190562' is not recognized as an internal or external command,
operable program or batch file.
WARNING: Error while running UEFIDump

Update of the default customizable variables to the latest link and more generic paths
Removal of training spaces

tandasat added 3 commits

December 28, 2020 13:27


          fix str vs byte mismatches

c785e3f


          remove trailing spaces


          fix os.system error on Windows

d9cbb0a

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

Labels

None yet