Comments (4)
Also I think we need to refactor the responsde code handling to make it less clunky:
let ret = Error::from_tss_rc(ret);
if ret.success() {
...
} else {
...
}
We should convert ret
to a Result<(), Error>
from rust-tss-esapi.
I am not sure. I was just unable check if I got the error that I was interested in and I did some minor digging in the specifications but did not really come up with any answers.
From reading the specification we can say for a fact that we are not reporting the layer in software stack from where the error originated. That could be information that is worth providing.
from rust-tss-esapi.
As in we cover the TSS2_RC
values but not the TSS2_ESYS_RC
ones? Going through the spec there are also the TSS2_TCTI_RC
values 😨
from rust-tss-esapi.
I have started to work on this and it is 'not to bad'. It seems as we are just missing a lot of information and in some cases when the error is originating from the software stack it self and in that case we might be doing something wrong. But the errors originating from the TPM are handled correct from what I can see.
But I will refactor the response code and add support for errors originating from the software stack. But this will probably require breaking the API.
from rust-tss-esapi.
Related Issues (20)
- Creating an ecc attestation key fails due to missing public HOT 1
- Windows TCTI/TBS support HOT 29
- No 7.3.0 tag in git HOT 2
- Support - help needed with key duplication for another example HOT 1
- Add CI job for examples
- Remove clippy checks from test scripts
- Incorrect handling of FFI `size_t` in marshalling and unmarshalling trait implementation.
- Set rust-version in Cargo.toml
- No way to sign without hashcheck ticket HOT 4
- how to use nv_read_public()? HOT 2
- The max size of the 'SensitiveData' buffer is to big.
- Non sized buffers are not checked for exact size.
- v8.0.0 stabilization work HOT 21
- Change minimum required tpm2-tss version.
- Idea: PublicBuilder presets for common usecases HOT 3
- Support Created Loaded HOT 5
- Update Rust edition.
- Add Ci for Windows
- Dependency mbox 0.6.1 which hard requires Rust 2018 HOT 2
- Make the serde dependency optional.
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from rust-tss-esapi.