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

scripts: imgtool: fix compression with encryption #394

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

michalek-no
Copy link
Contributor

adds TLV which stores compressed image size in case encryption is on. This is to avoid wrong streaam size due to encryption padding.

@michalek-no michalek-no force-pushed the mb-imgtool-enc-comp branch 3 times, most recently from 24b6889 to f9999b5 Compare January 9, 2025 08:28
@michalek-no michalek-no added the DNM label Jan 9, 2025
@michalek-no michalek-no force-pushed the mb-imgtool-enc-comp branch 3 times, most recently from 58ce51e to 009b3c8 Compare January 9, 2025 13:01
@michalek-no michalek-no removed the DNM label Jan 9, 2025
adds TLV which stores compressed image size in case encryption is on.
This is to avoid wrong streaam size due to encryption padding.

Upstream PR #: 2170

Signed-off-by: Mateusz Michalek <[email protected]>
(cherry picked from commit a44416620f8475692cc5509532801164e80fb50d)
…rypted image

This TLV is needed in order to know what the data length
provided to the decompression system is to remove the padding
that is a resultant of the encryption block size

Signed-off-by: Jamie McCrae <[email protected]>
(cherry picked from commit 4bda587549c0fccd5cd9b59d76c8edf061ee84c3)
@nvlsianpu
Copy link
Contributor

@michalek-no Are you going to fix git history issue signaled by the Commit tag?

@michalek-no
Copy link
Contributor Author

@michalek-no Are you going to fix git history issue signaled by the Commit tag?

This probably will be closed after upmerge as not needed. Otherwise yes.

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.

4 participants