Page MenuHomeFreeBSD
Differential D52438

nuageinit: chmod sudoers directory instead of chmod (again) sudoers file
ClosedPublic
Actions

Authored by dtxdf on Sep 8 2025, 10:23 PM.
Tags
None
Referenced Files
F154516577: D52438.id161753.diff
Tue, Apr 28, 7:34 PM
F154513581: D52438.id161753.diff
Tue, Apr 28, 7:27 PM
F154496242: D52438.id161766.diff
Tue, Apr 28, 6:27 PM
F154492930: D52438.id161766.diff
Tue, Apr 28, 6:19 PM
F154452082: D52438.id161835.diff
Tue, Apr 28, 4:02 PM
Unknown Object (File)
Tue, Apr 28, 2:05 AM
Unknown Object (File)
Mon, Apr 27, 8:01 PM
Unknown Object (File)
Mon, Apr 27, 2:53 PM
View All 89 Files
Subscribers
imp
jlduran

Details

Reviewers
bapt
jlduran
Commits
rG5d8efb8f2578: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file
rGa00410fa19e9: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file
rGa5cc9b7b96dc: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

dtxdf created this revision.Sep 8 2025, 10:23 PM
Herald added a subscriber: imp. · View Herald TranscriptSep 8 2025, 10:23 PM
dtxdf requested review of this revision.Sep 8 2025, 10:23 PM
jlduran added a subscriber: jlduran.Sep 9 2025, 4:33 AM
jlduran added inline comments.
libexec/nuageinit/nuage.lua
363

Should also be "0750".

dtxdf updated this revision to Diff 161766.Sep 9 2025, 6:02 AM
  • Set mode of sudoers directory to 0750
dtxdf marked an inline comment as done.Sep 9 2025, 6:03 AM
jlduran accepted this revision.Sep 9 2025, 3:06 PM
jlduran added inline comments.
libexec/nuageinit/nuage.lua
360

Not related to this change, but the permissions for sudoers should be 0440.

This revision is now accepted and ready to land.Sep 9 2025, 3:06 PM
jlduran added a comment.Sep 9 2025, 3:10 PM

For future reviews, please attach a full diff, in order to display the context (or better yet, use git-arc(1)).

bapt accepted this revision.Sep 9 2025, 3:11 PM
dtxdf updated this revision to Diff 161835.Sep 9 2025, 7:26 PM
  • Set mode of sudoers to 0440.
This revision now requires review to proceed.Sep 9 2025, 7:26 PM
jlduran accepted this revision.Sep 9 2025, 8:41 PM
This revision is now accepted and ready to land.Sep 9 2025, 8:41 PM
Closed by commit rGa5cc9b7b96dc: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file (authored by dtxdf). · Explain WhySep 11 2025, 5:08 PM
This revision was automatically updated to reflect the committed changes.
dtxdf added a commit: rGa5cc9b7b96dc: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file.
bapt added a commit: rGa00410fa19e9: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file.Sep 17 2025, 7:34 PM
bapt added a commit: rG5d8efb8f2578: nuageinit: chmod sudoers directory instead of chmod (again) sudoers file.Sep 17 2025, 7:42 PM

Revision Contents
Changeset List

PathSize
libexec/
nuageinit/
2 lines

Diff 161766

View Options

libexec/nuageinit/nuage.lua

Loading...