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

Hotfix functionality incompatible with composefs+verity #3187

Closed
rborn-tx opened this issue Feb 20, 2024 · 2 comments
Closed

Hotfix functionality incompatible with composefs+verity #3187

rborn-tx opened this issue Feb 20, 2024 · 2 comments

Comments

@rborn-tx
Copy link
Contributor

I noticed the hotfix functionality (related to ostree admin unlock --hotfix) is available even when one is using composefs (at least the part that applies the hotfix).

Specifically, in this stretch of code, ostree-prepare-root checks for the existence of a "hotfix overlay" by looking at the deployment directory even when composefs is in use. Unless I'm missing something, this would allow one to apply a hotfix on top of a composefs+verity image and hence bypass the integrity/authenticity guarantees you would normally expect. I suppose this is not intended, right?

Since the hotfix is made of overlay directories (workdir, upperdir) which aren't signed I suppose composefs and hotfixing are not compatible...

@alexlarsson
Copy link
Member

Yeah. I don't think should allow this for a signed composefs.

alexlarsson added a commit to alexlarsson/ostree that referenced this issue Feb 22, 2024
As mentioned in ostreedev#3187, we
can't allow a hotfix overlay of /usr when using signed composefs
images as that would allow an attacker to persist something used
across boots.
@ericcurtin ericcurtin mentioned this issue Feb 23, 2024
33 tasks
@cgwalters
Copy link
Member

Thanks for noticing this!

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

No branches or pull requests

4 participants