Skip to content

Commit

Permalink
certctl: Set METALOG ownership to root:wheel
Browse files Browse the repository at this point in the history
This sets the correct ownership values when building base.txz

This is an updated version of commit 10fa3f2, omitting the
`-N ${DESTDIR}${DISTBASE}/etc` from the original commit.

install(1) does not validate the arguments passed to -o or -g
(see PR283355) so there's no need to have the passwd db available
for now.  Future work includes plumbing the appropriate passwd db
path through certctl, and validating uid and gid in install(1).

PR:		283340
Reviewed by:	jrtc27
Differential Revision: https://reviews.freebsd.org/D48506
  • Loading branch information
patmaddox authored and emaste committed Jan 17, 2025
1 parent a8b49e7 commit 4d15b58
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion usr.sbin/certctl/certctl.sh
Original file line number Diff line number Diff line change
Expand Up @@ -334,7 +334,7 @@ fi
: ${METALOG:=${DESTDIR}/METALOG}
INSTALLFLAGS=
if "$UNPRIV" ; then
INSTALLFLAGS="-U -M ${METALOG} -D ${DESTDIR}"
INSTALLFLAGS="-U -M ${METALOG} -D ${DESTDIR} -o root -g wheel"
fi
: ${LOCALBASE:=$(sysctl -n user.localbase)}
: ${TRUSTPATH:=${DESTDIR}${DISTBASE}/usr/share/certs/trusted:${DESTDIR}${LOCALBASE}/share/certs:${DESTDIR}${LOCALBASE}/etc/ssl/certs}
Expand Down

0 comments on commit 4d15b58

Please sign in to comment.