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.

Sign up for GitHub

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

Jump to bottom

Add check for nosuid data mount #592

Merged
merged 1 commit into from
Oct 16, 2024
Merged

Add check for nosuid data mount #592

merged 1 commit into from
Oct 16, 2024

Conversation

spencerpogo
Copy link
Contributor

Don't want this to happen to anyone else lol

@zardus
Copy link
Contributor

zardus commented Oct 15, 2024

This doesn't add any requirements on /data, right? We already need that to be a mount point?

@spencerpogo
Copy link
Contributor Author

Yes, exactly. The findmnt -n check was already there. It will always be a mount point because docker will have to create one there.

@zardus zardus merged commit f36be4e into pwncollege:master Oct 16, 2024
1 check passed
@zardus
Copy link
Contributor

zardus commented Oct 16, 2024

Thanks!

@spencerpogo spencerpogo deleted the check-data-nosuid branch October 16, 2024 05:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@spencerpogo @zardus