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

Restrict named mutex files permissions #78106

Merged
merged 1 commit into from
Nov 10, 2022

Conversation

janvorli
Copy link
Member

@janvorli janvorli commented Nov 9, 2022

This change restricts the permission for files created for named mutexes that are not machine wide. Until now, all named mutexes had underlying files with access to all users. With this change, mutexes that are session local have access for the current user only. In addition to that, sticky bit is set for the /tmp/.dotnet, /tmp/.dotnet/shm directories to ensure that only creator of a file or a subdirectory can delete the respective file / subdirectory.

This change restricts the permission for files created for named mutexes
that are not machine wide. Until now, all named mutexes had
underlying files with access to all users. With this change,
mutexes that are session local have access for the current user only.
In addition to that, sticky bit is set for the /tmp/.dotnet,
/tmp/.dotnet/shm directories to ensure that only creator of a file
or a subdirectory can delete the respective file / subdirectory.
@janvorli janvorli added this to the 8.0.0 milestone Nov 9, 2022
@janvorli janvorli requested a review from jkotas November 9, 2022 15:28
@janvorli janvorli self-assigned this Nov 9, 2022
@janvorli janvorli merged commit ae1bff0 into dotnet:main Nov 10, 2022
@janvorli janvorli deleted the modify-mutex-files-permissions branch November 10, 2022 00:13
@ghost ghost locked as resolved and limited conversation to collaborators Dec 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants