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

Validate SNP Attestation Report Fields #15

Open
npmccallum opened this issue Mar 7, 2022 · 4 comments
Open

Validate SNP Attestation Report Fields #15

npmccallum opened this issue Mar 7, 2022 · 4 comments
Labels
attestation Issues related to attestation

Comments

@npmccallum
Copy link
Member

The fields are documented on page 44 of the SNP Firmware ABI spec. They are implemented in this structure, but some fields appear to be missing and nearly all appear to be undocumented.

  1. All fields from the documentation need to be accounted for (some are currently missing).
  2. All fields need to be documented in the code, explaining how to validate them or why validation is unnecessary.
  3. Fields that represent flags should probably use the flagset crate.
  4. Whoever takes this should work with @haraldh to confirm which fields need to be validated.
  5. Implement all necessary validations.
@npmccallum
Copy link
Member Author

cc @rjzak

@rjzak
Copy link
Member

rjzak commented Mar 9, 2022

  • All fields from the documentation need to be accounted for (some are currently missing).
  • All fields need to be documented in the code, explaining how to validate them or why validation is unnecessary.
    • Most fields documented as of commit 319b24f, but could be more detailed.
    • In the future, may want to validate against firmware versions.
  • Fields that represent flags should probably use the flagset crate.
  • Whoever takes this should work with @haraldh to confirm which fields need to be validated.
  • Implement all necessary validations.
    • Probably should allow the last reserved field to have fields, as future revisions to the firmware may add fields, possibly before Steward can be updated to accommodate.

@rjzak
Copy link
Member

rjzak commented Apr 19, 2022

I believe the SNP fields have been validated.

@rjzak rjzak closed this as completed Apr 19, 2022
@haraldh
Copy link
Member

haraldh commented Oct 28, 2022

we might want to reopen this

@rjzak rjzak reopened this Oct 28, 2022
@dpal dpal added the attestation Issues related to attestation label Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
attestation Issues related to attestation
Projects
None yet
Development

No branches or pull requests

4 participants