Page MenuHomeFreeBSD
Differential D17973

Fix /etc/ntp mode
ClosedPublic
Actions

Authored by garga on Nov 13 2018, 12:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 7, 12:33 AM
Unknown Object (File)
Wed, Dec 31, 10:28 AM
Unknown Object (File)
Mon, Dec 15, 12:01 AM
Unknown Object (File)
Sat, Dec 13, 11:41 AM
Unknown Object (File)
Nov 28 2025, 4:46 PM
Unknown Object (File)
Nov 25 2025, 9:06 PM
Unknown Object (File)
Nov 14 2025, 9:56 AM
Unknown Object (File)
Nov 14 2025, 12:39 AM
View All 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...