Page MenuHomeFreeBSD
Differential D17973

Fix /etc/ntp mode
ClosedPublic
Actions

Authored by garga on Nov 13 2018, 12:39 PM.
Tags
None
Referenced Files
F135943092: D17973.id50418.diff
Fri, Nov 14, 9:56 AM
Unknown Object (File)
Fri, Nov 14, 12:39 AM
Unknown Object (File)
Sat, Nov 8, 4:49 AM
Unknown Object (File)
Sun, Nov 2, 8:16 AM
Unknown Object (File)
Fri, Oct 24, 5:39 AM
Unknown Object (File)
Fri, Oct 24, 5:39 AM
Unknown Object (File)
Fri, Oct 24, 5:38 AM
Unknown Object (File)
Thu, Oct 23, 8:50 PM
View All 98 Files
Subscribers
imp
yuripv

Details

Reviewers
brd
imp
Commits
rS340439: Fix /etc/ntp permissions. According to mtree it must be 0700
Summary

According to BSD.root.dist /etc/ntp should be 0700 but it's being created
as 0755 and every time installworld runs I see the following message:

etc/ntp:
        permissions (0700, 0755, modified)
crash:  permissions (0750, 0755, modified)

Change ntpd Makefile to make sure 0700 is used

MFC after: 3 days

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

garga created this revision.Nov 13 2018, 12:39 PM
Herald added a subscriber: imp. · View Herald TranscriptNov 13 2018, 12:39 PM
Harbormaster completed remote builds in B20783: Diff 50361.Nov 13 2018, 12:39 PM
garga edited the summary of this revision. (Show Details)Nov 13 2018, 12:40 PM
imp accepted this revision.Nov 13 2018, 2:46 PM
This revision is now accepted and ready to land.Nov 13 2018, 2:46 PM
garga added a comment.Nov 14 2018, 11:01 AM

Since I don't have a src bit, may I add "Approved by: imp"?

imp added a comment.Nov 14 2018, 3:39 PM
In D17973#384258, @garga wrote:

Since I don't have a src bit, may I add "Approved by: imp"?

Yes.

I assume that you've run buildworld / installworld and there's no ill effects from this and the files wind up having the proper permissions / ownership?

Closed by commit rS340439: Fix /etc/ntp permissions. According to mtree it must be 0700 (authored by garga). · Explain WhyNov 14 2018, 4:19 PM
This revision was automatically updated to reflect the committed changes.
yuripv added a subscriber: yuripv.Nov 14 2018, 4:31 PM
yuripv added inline comments.
head/usr.sbin/ntp/ntpd/Makefile
12

Is ETN a typo or intentional?

garga marked an inline comment as done.Nov 14 2018, 6:11 PM
garga added inline comments.
head/usr.sbin/ntp/ntpd/Makefile
12

typo, I'll commit a fix. Thanks!

Revision Contents
Changeset List

PathSize
head/
usr.sbin/
ntp/
ntpd/
5 lines

Diff 50418

View Options

head/usr.sbin/ntp/ntpd/Makefile

Loading...