Skip to content

chown: Replace unsafe calls with rustix (Issue: #12041)#12107

Draft
ksgk1 wants to merge 3 commits intouutils:mainfrom
ksgk1:replace-unsafe-chown-by-rustix-#12041
Draft

chown: Replace unsafe calls with rustix (Issue: #12041)#12107
ksgk1 wants to merge 3 commits intouutils:mainfrom
ksgk1:replace-unsafe-chown-by-rustix-#12041

Conversation

@ksgk1
Copy link
Copy Markdown
Contributor

@ksgk1 ksgk1 commented May 1, 2026

In this PR, I am replacing the unsafe calls for libc with rustix for chown and chown_at for issue #12041.

@cakebaker cakebaker linked an issue May 1, 2026 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

GNU testsuite comparison:

GNU test failed: tests/chgrp/basic. tests/chgrp/basic is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/chgrp/from. tests/chgrp/from is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/chown/basic. tests/chown/basic is passing on 'main'. Maybe you have to rebase?
Congrats! The gnu test tests/seq/seq-epipe is no longer failing!
Note: The gnu test tests/chgrp/deref is now being skipped but was previously passing.
Note: The gnu test tests/tail/tail-n0f is now being skipped but was previously passing.
Congrats! The gnu test tests/pr/bounded-memory is now passing!
Note: The gnu test tests/env/env-signal-handler was skipped on 'main' but is now failing.

@ksgk1 ksgk1 marked this pull request as draft May 1, 2026 10:39
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.

Some unsafe chown(at) code can be removed by rustix

1 participant