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

Add marshalling to Private (TPM2B_PRIVATE) #457

Merged

Conversation

brandsimon
Copy link
Contributor

Good day,

I need this to load an object created with tpm2_create.

@brandsimon brandsimon force-pushed the sbr/add_public_marshalling branch from c961edf to 0d5e37c Compare October 13, 2023 23:15
@Superhepper
Copy link
Collaborator

Thanks for the PR. It would also be great if you could add some tests for this as well.

@brandsimon brandsimon force-pushed the sbr/add_public_marshalling branch from e3a0a7f to d9ba0d9 Compare October 15, 2023 22:38
@brandsimon
Copy link
Contributor Author

@Superhepper good catch, done

@Superhepper
Copy link
Collaborator

Sorry for not noticing this the first time but it is better to implement the marshall_offset and unmarshall_offset because the default implementation of marshall calls marshall_offset with offset=0.

Copy link
Member

@ionut-arm ionut-arm left a comment

Choose a reason for hiding this comment

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

Thanks for the patch!!

@brandsimon brandsimon force-pushed the sbr/add_public_marshalling branch from 46a3d9d to 7190f3d Compare October 17, 2023 23:16
@brandsimon
Copy link
Contributor Author

@Superhepper no worries, code looks cleaner with the _offset methods, I like it! :)

@brandsimon brandsimon force-pushed the sbr/add_public_marshalling branch from ad16e07 to 8066e05 Compare October 19, 2023 23:06
@ionut-arm ionut-arm merged commit d942908 into parallaxsecond:main Oct 21, 2023
10 checks passed
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.

3 participants