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

feat(sandbox): add remove_dir permissions to sandbox API #82

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

qkaiser
Copy link
Contributor

@qkaiser qkaiser commented Jan 22, 2025

When an extraction directory is empty, unblob will try to delete it. This can lead to PermissionError due to insufficient permissions within the sandbox.

We therefore expose the RemoveDir permission through the sandbox API so it can be used by unblob.

@qkaiser qkaiser added the bug Something isn't working label Jan 22, 2025
@qkaiser qkaiser requested a review from vlaci January 22, 2025 10:16
@qkaiser qkaiser self-assigned this Jan 22, 2025
@qkaiser qkaiser force-pushed the landlock-extension branch from 6fba785 to f302521 Compare January 22, 2025 10:17
@qkaiser
Copy link
Contributor Author

qkaiser commented Jan 22, 2025

CI fails on aarch64 because of this: #83

@qkaiser qkaiser force-pushed the landlock-extension branch from f302521 to da77540 Compare January 23, 2025 16:33
When an extraction directory is empty, unblob will try to delete it.
This can lead to PermissionError due to insufficient permissions within
the sandbox.

We therefore expose the RemoveDir and RemoveFile permissions through the
sandbox API so it can be used by unblob.
@qkaiser qkaiser force-pushed the landlock-extension branch from da77540 to b90dfb0 Compare January 23, 2025 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants