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

copy_tree: Fix regressions #565

Merged
merged 2 commits into from
Sep 9, 2022
Merged

Conversation

ferivoz
Copy link
Contributor

@ferivoz ferivoz commented Sep 4, 2022

This pull request fixes regressions in copy_tree:

  • ./configure --without-acl leads to a usermod which does not work due to invalid chmod call.
  • FIFOs in user home directories prevent usermod -md /new_home user from working.

Please see commit messages for more details.

Fixes regression introduced in faeab50
for setups configured without acl support.
Fixes regression introduced in faeab50.

If a directory contains fifos, then openat blocks until the other side
of the fifo is connected as well.

This means that users can prevent "usermod -m" from completing if their
home directories contain at least one fifo.
Copy link
Collaborator

@ikerexxe ikerexxe left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the patches!

@ikerexxe ikerexxe merged commit 10cd68e into shadow-maint:master Sep 9, 2022
@rbalint
Copy link
Contributor

rbalint commented Nov 7, 2022

@hallyn It would be nice to have tagged releases when similar regressions are fixed. Most likely distros would like to fix such regressions ASAP.

@hallyn
Copy link
Member

hallyn commented Nov 7, 2022

Oh, has it been that long? Sure, I'll do a new release.

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this pull request Nov 9, 2022
https://build.opensuse.org/request/show/1034152
by user jubalh + dimstar_suse
- Add shadow-copytree-usermod-fifo.patch:
  Fix regression that prevented `usermod -m` to work when their
  home directory contained at least one fifo
  See shadow-maint/shadow#565
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.

5 participants