Page MenuHomeFreeBSD
Differential D45098

adduser: Fix confusion between `uclass` and `_class`.
ClosedPublic
Actions

Authored by des on Mon, May 6, 12:00 PM.
Tags
None
Referenced Files
F84130586: D45098.diff
Sun, May 19, 7:11 PM
Unknown Object (File)
Tue, May 14, 3:06 PM
Unknown Object (File)
Thu, May 9, 6:22 PM
Unknown Object (File)
Thu, May 9, 4:28 PM
Unknown Object (File)
Thu, May 9, 4:28 PM
Unknown Object (File)
Wed, May 8, 6:06 PM
Unknown Object (File)
Wed, May 8, 6:36 AM
Unknown Object (File)
Wed, May 8, 6:28 AM
View All 15 Files
Subscribers
None

Details

Reviewers
karels
imp
allanjude
Commits
rG01872b366578: adduser: Fix confusion between `uclass` and `_class`.
rG9f21d0ef9196: adduser: Fix confusion between `uclass` and `_class`.
rGe121845426c8: adduser: Fix confusion between `uclass` and `_class`.
rG46c4e86ebf18: adduser: Fix confusion between `uclass` and `_class`.
Summary

This caused adduser to produce an invalid pw(8) command line. Due to
bugs in pw(8), the command line was silently accepted and led to the
user being created, but locked out and with no home directory.

Fixes: 170d0882903e
MFC after: 3 days

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

des requested review of this revision.Mon, May 6, 12:00 PM
des created this revision.
Harbormaster completed remote builds in B57593: Diff 138157.Mon, May 6, 12:00 PM
des added a parent revision: D45097: pw: Don't silently ignore unparsed command line arguments..Mon, May 6, 12:00 PM
des updated this revision to Diff 138158.Mon, May 6, 12:45 PM

move _another_user

Harbormaster completed remote builds in B57594: Diff 138158.Mon, May 6, 12:45 PM
karels added a comment.Mon, May 6, 12:48 PM

I may not be the best reviewer for this change; I haven't used "local".

allanjude accepted this revision.Mon, May 6, 1:11 PM
This revision is now accepted and ready to land.Mon, May 6, 1:11 PM
karels added a comment.Mon, May 6, 1:21 PM

I applied this change (only) and ran the script; I got:

OK? (yes/no) [yes]:
adduser.fix: INFO: Successfully added (mike) to the user database.
local: Not in a function
Add another user? (yes/no) []:

des added a comment.Mon, May 6, 1:22 PM
In D45098#1028400, @karels wrote:

I may not be the best reviewer for this change; I haven't used "local".

Perhaps, but you reported the bug and you can at least test the patch.

des updated this revision to Diff 138159.Mon, May 6, 1:23 PM

not in a function

This revision now requires review to proceed.Mon, May 6, 1:23 PM
Harbormaster completed remote builds in B57595: Diff 138159.Mon, May 6, 1:23 PM
karels added a comment.Mon, May 6, 1:43 PM

This version works (adds the user correctly and defaults to not adding another user). Tested only on UFS; will test on ZFS in an hour or so.

karels added a comment.Mon, May 6, 2:22 PM

Works on ZFS too; dataset added and home directory set up correctly.

des added a comment.Mon, May 6, 2:38 PM

Can you please approve it, then?

karels accepted this revision.Mon, May 6, 3:26 PM

Accepted, based mostly on Alan's previous approval.

This revision is now accepted and ready to land.Mon, May 6, 3:26 PM
Closed by commit rG46c4e86ebf18: adduser: Fix confusion between `uclass` and `_class`. (authored by des). · Explain WhyMon, May 6, 4:20 PM
This revision was automatically updated to reflect the committed changes.
des added a commit: rG46c4e86ebf18: adduser: Fix confusion between `uclass` and `_class`..
des added a commit: rGe121845426c8: adduser: Fix confusion between `uclass` and `_class`..Thu, May 9, 1:16 PM
des added a commit: rG9f21d0ef9196: adduser: Fix confusion between `uclass` and `_class`..
des added a commit: rG01872b366578: adduser: Fix confusion between `uclass` and `_class`..Thu, May 9, 2:32 PM

Revision Contents
Changeset List

PathSize
usr.sbin/
adduser/
6 lines

Diff 138166

View Options

usr.sbin/adduser/adduser.sh

Loading...