-
Notifications
You must be signed in to change notification settings - Fork 16
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
Extend SGX-support to describe other properties without relying on OE/Gramine-verify. #175
Comments
Open Enclave claims are extracted from evidence/endorsement as part of the verifier interfaces. The claims are returned by the verifier as an array of oe_claim_t elements:
Optional claim reliability is questionable. Especially across OE SDK versions. More claims were added in later versions. |
Revisited the scope of this issue: As there seems to be some ambiguity w.r.t how-to get all the attributes across diff OE-versions, we are leaning towards working-in support for Gramine first. John & Ye will re-examine use of DCAP interfaces , instead, to get this info. |
This is complete for Gramine. |
9/25/2024: Revisited ... low-priority; Keep this open for now. Unlike the things discussed in #87, this item will is SGX-only and will not apply to TDX. |
When we do verification on SGX platform, we need enhancements to return other 'collateral' (to be specified).
in SEV, we check platform properties, e.g. if debug is turned ON, if you can share keys w/ other platforms, API u-code level etc.
We don't do that for SGX. We only check is the measurement. Reason we don't check is : That was not one of the things returned by OE-verify, Gramine-verify (on SGX platform).
We don't check SGX-report directly. We rely on OE/Gramine to do some of the verification / signature(s).
We don't rely on them to check the measurements; we do that directly.
We should directly check the SGX-properties (e.g. debug, whether this enclave allows other enclaves to debug it, TCB-version etc ...) ourselves.
Dependencies: Ye has to do something first, and then John can work on this.
The text was updated successfully, but these errors were encountered: