diff options
author | Ed Maste <emaste@FreeBSD.org> | 2023-09-16 20:46:16 +0000 |
---|---|---|
committer | Ed Maste <emaste@FreeBSD.org> | 2023-09-21 14:40:03 +0000 |
commit | 877d15d57931350dcde264de1dabe7fccf769388 (patch) | |
tree | 1bd9445fef66fb64dde556d079ed3e5a09bce3f0 | |
parent | a4916232acd614d4d7d7226d678b427aacb9730e (diff) |
freebsd-update: do not restart sshd when updating jail/basedir
In 6cd1bc531609 for PR 263489 I changed freebsd-update to restart sshd
after upgrade, to avoid an upgrade-related incompatibility that made it
impossible to login.
This is intended to avoid losing access to remote hosts, and ought not
apply to upgrading jails (from outside).
PR: 263489, 272282
Reported by: otis
Reviewed by: otis, kevans
MFC after: 3 days
Sponsored by: The FreeBSD Foundation
Fixes: 6cd1bc531609 ("freebsd-update: restart sshd after upgrade")
Differential Revision: https://reviews.freebsd.org/D41890
(cherry picked from commit 8ee97b1928e75f7f81a870ffb063010778e0a129)
(cherry picked from commit cce342e21357833892083e865710c4a05603b7b5)
-rw-r--r-- | usr.sbin/freebsd-update/freebsd-update.sh | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/usr.sbin/freebsd-update/freebsd-update.sh b/usr.sbin/freebsd-update/freebsd-update.sh index 57023557839b..9cdff05c24e4 100644 --- a/usr.sbin/freebsd-update/freebsd-update.sh +++ b/usr.sbin/freebsd-update/freebsd-update.sh @@ -3036,9 +3036,10 @@ Kernel updates have been installed. Please reboot and run install_from_index INDEX-NEW || return 1 install_delete INDEX-OLD INDEX-NEW || return 1 - # Restart sshd if running (PR263489). Note that this does not - # affect child sshd processes handling existing sessions. - if service sshd status >/dev/null 2>/dev/null; then + # Restart host sshd if running (PR263489). Note that this does + # not affect child sshd processes handling existing sessions. + if [ "$BASEDIR" = / ] && \ + service sshd status >/dev/null 2>/dev/null; then echo echo "Restarting sshd after upgrade" service sshd restart |