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

Persistent objects: save handle information #795

Merged
1 commit merged into from
May 20, 2016
Merged

Persistent objects: save handle information #795

1 commit merged into from
May 20, 2016

Conversation

pascal-brand38
Copy link
Contributor

When creating a Persistent Object (TEE_CreatePersistentObject)
using an object handle, the information and attributes related
to the handle are saved.

The information are restored when opening the object
(TEE_OpenPersistentObject)

Fixes #785

Signed-off-by: Pascal Brand pascal.brand@st.com

@@ -77,6 +77,8 @@ struct tee_svc_storage_head {
uint32_t head_size;
uint32_t meta_size;
uint32_t ds_size;
TEE_ObjectInfo info;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This saves redundant or unneeded information in the persistent object. I would prefer to just add the needed fields from TEE_ObjectInfo.

@pascal-brand38
Copy link
Contributor Author

updated

@jenswi-linaro
Copy link
Contributor

Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>

When creating a Persistent Object (TEE_CreatePersistentObject)
using an object handle, the information and attributes related
to the handle are saved.

The information are restored when opening the object
(TEE_OpenPersistentObject)

Fixes #785

Reviewed-by: Jens Wiklander <jens.wiklander@linaro.org>
Signed-off-by: Pascal Brand <pascal.brand@st.com>
@ghost ghost merged commit 50ffad3 into OP-TEE:master May 20, 2016
@pascal-brand38 pascal-brand38 deleted the persistent branch May 20, 2016 07:43
This pull request was closed.
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