passwd after password db changes
ClosedPublic
Actions

Authored by aryeeteygerald_rogers.com on Jan 11 2019, 10:24 PM.

Details

Reviewers

emaste
jilles

Commits

rS343270: freebsd-update: Update /etc/passwd after password db changes

Summary

Add -p to pwd_mkdb in order to ensure password db
changes are also included in /etc/passwd.

PR: 165954, 232921, +

Test Plan

None

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 22008
Build 21242: arc lint + arc unit

Event Timeline

aryeeteygerald_rogers.com created this revision.Jan 11 2019, 10:24 PM

Harbormaster completed remote builds in B21936: Diff 52787.Jan 11 2019, 10:24 PM

aryeeteygerald_rogers.com edited the summary of this revision. (Show Details)Jan 15 2019, 10:28 PM

emaste added subscribers: cperciva, gordon.Jan 16 2019, 10:00 PM

emaste added a subscriber: bcran.Jan 17 2019, 7:20 PM

emaste added inline comments.Jan 17 2019, 7:23 PM

usr.sbin/freebsd-update/freebsd-update.sh
2899	This comment is out of date now - maybe just "Rebuild generated pwd files."

Update outdated comment

Harbormaster completed remote builds in B22008: Diff 52940.Jan 17 2019, 7:30 PM

I wonder when it is appropriate to run pwd_mkdb without -p. Only when etc/passwd is considered legacy and needs to be omitted from the system?

This revision is now accepted and ready to land.Jan 18 2019, 2:36 PM

aryeeteygerald_rogers.com marked an inline comment as done.Jan 18 2019, 5:00 PM

Closed by commit rS343270: freebsd-update: Update /etc/passwd after password db changes (authored by emaste). · Explain WhyJan 21 2019, 7:04 PM

This revision was automatically updated to reflect the committed changes.

Herald added a subscriber: imp. · View Herald TranscriptJan 21 2019, 7:04 PM

Revision Contents
Changeset List

Path

Size

usr.sbin/

freebsd-update/

freebsd-update.sh

7 lines

Diff 52940

View Options

usr.sbin/freebsd-update/freebsd-update.sh

Update /etc/passwd after password db changesClosedPublicActions