Rpmb fix #1921
Merged
Rpmb fix #1921
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Fixes #1780 |
|
For "core: RPMB_FS: remember owner uuid", with the proper 12-character SHA1 prefix in the commit description: |
|
FYI, "core: RPMB_FS: remember owner uuid" has a proper 12-character SHA1 prefix in the commit description, it's just github mangling the commit message. |
|
Ah, yes indeed, but checkpatch expects "commit 078f18f ..." (the word "commit" has to be there). |
|
Message updated, tag applied |
|
#1919 is updated (tag applied) |
|
@jenswi-linaro can you please rebase now that 1919 is merged? Thanks. |
Prior to this patch was the owning uuid looked up via the current session. The assumption that the uuid of the context of the current session is the same as the owner of a RPMB file doesn't hold any longer after: commit 078f18f ("core: RPMB FS: provide tee_rpmb_fs_raw_open()") was introduced. Instead the pointer to the uuid passed as the owner is stored in the file handle and used as needed when encrypting/decrypting the file. Acked-by: Jerome Forissier <[email protected]> Fixes: OP-TEE#1780 Tested-by: Jens Wiklander <[email protected]> (QEMU) Signed-off-by: Jens Wiklander <[email protected]>
|
Rebased |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For all tests to pass #1919 is needed too, please ignore
"core: fix access conflict status in rpmb fs that panics TA" while reviewing this PR