Skip to content

Commit

Permalink
Allow logging in to rescue/emergency shell as root
Browse files Browse the repository at this point in the history
In Qubes OS guests, the console is _always_ the Xen console.  There are
_never_ untrusted entities with access to this under _any_
circumstances.  The only supported way to get access to this from a VM
is the admin.vm.Console service, which is (by design) equivalent to root
access in the destination VM.  Other consoles, such as a USB serial
console, are not supported as domU consoles under Qubes OS.

Therefore, set SYSTEMD_SULOGIN_FORCE=1 to tell systemd-sulogin-shell to
pass --force to sulogin(8).  Since the root account is locked in Qubes
VMs, this causes sulogin(8) to allow login as root without a password.
Use TTYPath= to enforce that the console is, in fact, the Xen console
and not some other console, in case the assumption in the previous
paragraph turns out false for some reason.  This will break if the Xen
drivers are not included in the guest kernel config, but an HVM guest
with Qubes tools but no Xen drivers is not a supported configuration.
  • Loading branch information
DemiMarie committed Dec 28, 2024
1 parent df1ffd4 commit f9c1fdf
Show file tree
Hide file tree
Showing 8 changed files with 60 additions and 3 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ SYSTEM_DROPINS += systemd-random-seed.service
SYSTEM_DROPINS += tor.service [email protected]
SYSTEM_DROPINS += systemd-timesyncd.service
SYSTEM_DROPINS += systemd-logind.service
SYSTEM_DROPINS += sysinit.target
SYSTEM_DROPINS += sysinit.target emergency.service rescue.service
ifeq ($(ENABLE_SELINUX),1)
SYSTEM_DROPINS += selinux-autorelabel.target selinux-autorelabel.service
endif
Expand Down
5 changes: 4 additions & 1 deletion boot/Makefile
Original file line number Diff line number Diff line change
@@ -1,11 +1,14 @@
LIBDIR ?= /usr/lib
SYSCONFDIR ?= /etc
GRUBCONFDIR ?= $(SYSCONFDIR)/default
DRACUTCONFDIR ?= $(LIBDIR)/dracut/dracut.conf.d
GRUBCFGNAME ?= grub.qubes
DRACUTDIR ?= $(LIBDIR)/dracut
DRACUTCONFDIR ?= $(DRACUTDIR)/dracut.conf.d
DRACUTMODDIR ?= $(DRACUTDIR)/modules.d

.PHONY: install

install:
install -D -m 0644 grub.qubes $(DESTDIR)$(GRUBCONFDIR)/$(GRUBCFGNAME)
install -D -m 0644 dracut-qubes.conf $(DESTDIR)$(DRACUTCONFDIR)/30-qubes.conf
install -D -m 0755 module-setup.sh $(DESTDIR)$(DRACUTMODDIR)/99qubes/module-setup.sh
4 changes: 4 additions & 0 deletions boot/dracut-qubes.conf
Original file line number Diff line number Diff line change
Expand Up @@ -19,5 +19,9 @@ omit_dracutmodules+=" tpm2-tss "
# drivers, and none of the boot screen is visible in a qube anyway
omit_dracutmodules+=" plymouth "

# Include Qubes OS-specific module to enable passwordless login on the Xen
# console as root.
force_add_dracutmodules+=" qubes "

# Exclude microcode, as VM cannot load it anyway, saves over 10MB of space
early_microcode="no"
10 changes: 10 additions & 0 deletions boot/module-setup.sh
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
depends () {
}

install () {
if [ -h /lib ]; then
inst_multiple /usr/lib/systemd/system/rescue.service.d/30_qubes.conf /usr/lib/systemd/system/emergency.service.d/30_qubes.conf
else
inst_multiple /lib/systemd/system/rescue.service.d/30_qubes.conf /lib/systemd/system/emergency.service.d/30_qubes.conf
fi
}
3 changes: 3 additions & 0 deletions debian/qubes-core-agent.install
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,8 @@ lib/systemd/system/sysinit.target.d/30_qubes.conf
lib/systemd/system/systemd-timesyncd.service.d/30_qubes.conf
lib/systemd/system/systemd-logind.service.d/30_qubes.conf
lib/systemd/resolved.conf.d/30_resolved-no-mdns-or-llmnr.conf
lib/systemd/system/rescue.service.d/30_qubes.conf
lib/systemd/system/emergency.service.d/30_qubes.conf
usr/lib/sysctl.d/20-qubes-core.conf
usr/lib/systemd/user/tracker-extract-3.service.d/30_qubes.conf
usr/lib/systemd/user/tracker-miner-fs-3.service.d/30_qubes.conf
Expand Down Expand Up @@ -139,6 +141,7 @@ usr/bin/qvm-connect-tcp
usr/bin/qvm-sync-clock
usr/lib/environment.d/60-gnome-software-fix.conf
usr/lib/dracut/dracut.conf.d/30-qubes.conf
usr/lib/dracut/modules.d/99qubes/module-setup.sh
usr/lib/python3/dist-packages/qubesagent-*.egg-info/*
usr/lib/python3/dist-packages/qubesagent/*
usr/lib/qubes-bind-dirs.d/30_cron.conf
Expand Down
29 changes: 28 additions & 1 deletion rpm_spec/core-agent.spec.in
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ Requires: sed
Requires: util-linux
Requires: e2fsprogs
Requires: hostname
Requires: (%{name}-dracut if dracut)
# for Qubes Manager VM updater
Requires: xterm
# for qubes-desktop-run
Expand Down Expand Up @@ -373,6 +374,19 @@ switching from user to root. Since all the user data in a VM is accessible
already from normal user account, there is not much more to guard there. Qubes
VMs are single user systems.

%package dracut
Summary: Dracut configuration needed in Qubes OS
License: GPL
Group: Qubes
Requires: dracut
Requires: qubes-core-agent = %{version}

%description dracut
Dracut configuration useful for Qubes OS. This enables
passwordless login in rescue and emergency modes. Unlike the
configuration in qubes-kernel-vm-support, it isn't needed to
boot a VM, but it does make recovery of broken VMs easier.

%package thunar
Summary: Thunar support for Qubes VM tools
Requires: Thunar
Expand Down Expand Up @@ -521,6 +535,10 @@ sed -i \
$RPM_BUILD_ROOT/etc/yum.repos.d/qubes-*.repo
%endif

install -D -m 0644 boot/dracut-qubes.conf $RPM_BUILD_ROOT/usr/lib/dracut/dracut.conf.d/30-qubes.conf
install -D -m 0644 boot/grub.qubes $RPM_BUILD_ROOT/etc/default/grub.qubes
install -D -m 0755 boot/module-setup.sh $RPM_BUILD_ROOT/usr/lib/dracut/modules.d/99qubes/module-setup.sh

%if ! %with_sysvinit
rm -rf $RPM_BUILD_ROOT/etc/init.d/qubes-* $RPM_BUILD_ROOT/etc/sysconfig/modules/qubes-core.modules
%endif
Expand Down Expand Up @@ -1026,7 +1044,7 @@ rm -f %{name}-%{version}
/usr/lib/qubes/resize-rootfs
/usr/lib/qubes/set-default-text-editor
/usr/lib/qubes/tinyproxy-wrapper
/usr/lib/dracut/dracut.conf.d/30-qubes.conf

/usr/lib/environment.d/60-gnome-software-fix.conf
%dir /usr/lib/qubes/init
/usr/lib/qubes/init/bind-dirs.sh
Expand Down Expand Up @@ -1096,6 +1114,11 @@ rm -f %{name}-%{version}
/usr/share/caja-python/extensions/qvm_dvm_caja.py*
%endif

%files dracut
/usr/lib/dracut/dracut.conf.d/30-qubes.conf
%dir /usr/lib/dracut/modules.d/99qubes
/usr/lib/dracut/modules.d/99qubes/module-setup.sh

%files nautilus
/usr/share/nautilus-python/extensions/qvm_copy_nautilus.py*
/usr/share/nautilus-python/extensions/qvm_move_nautilus.py*
Expand Down Expand Up @@ -1304,6 +1327,10 @@ The Qubes core startup configuration for SystemD init.
%dir %_unitdir/sysinit.target.d
%_unitdir/sysinit.target.d/30_qubes.conf
%dir %_userunitdir/*.service.d
%dir %_unitdir/rescue.service.d
%dir %_unitdir/emergency.service.d
%_unitdir/emergency.service.d/30_qubes.conf
%_unitdir/rescue.service.d/30_qubes.conf
%_userunitdir/tracker-extract-3.service.d/30_qubes.conf
%_userunitdir/tracker-miner-fs-3.service.d/30_qubes.conf
%_userunitdir/tracker-miner-fs-control-3.service.d/30_qubes.conf
Expand Down
5 changes: 5 additions & 0 deletions vm-systemd/emergency.service.d/30_qubes.conf
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
[Service]
# Ensure that the console is the secure Xen console,
# not e.g. a serial console that is exposed to the outside world.
TTYPath=/dev/hvc0
Environment=SYSTEMD_SULOGIN_FORCE=1
5 changes: 5 additions & 0 deletions vm-systemd/rescue.service.d/30_qubes.conf
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
[Service]
# Ensure that the console is the secure Xen console,
# not e.g. a serial console that is exposed to the outside world.
TTYPath=/dev/hvc0
Environment=SYSTEMD_SULOGIN_FORCE=1

0 comments on commit f9c1fdf

Please sign in to comment.