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

managarm: send mode in openat to posix #920

Merged
merged 4 commits into from
Oct 7, 2023

Conversation

ElectrodeYT
Copy link
Member

@ElectrodeYT ElectrodeYT commented Oct 7, 2023

Fix a bug where if a file is crated with open or openat, it will not have any permission modes.

More importantly, this removes about 50% of log spam.

Managarm PR: managarm#568

Copy link
Member

@no92 no92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

This error only has a defined value for GNU/Hurd, however it some systems (such as managarm) may find it useful to return internal errors.
At the moment, this error is only returned if the chmod that is fired when O_CREAT is used failes.
@qookei qookei merged commit f5e1b11 into managarm:master Oct 7, 2023
24 checks passed
@ElectrodeYT ElectrodeYT deleted the openat_fix_1 branch October 7, 2023 13:26
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.

3 participants