]> git.ipfire.org Git - thirdparty/systemd.git/commit
update-utmp: fix assertion failure if rescue.target, multi-user.target and graphical...
authorTommi Rantala <tommi.t.rantala@nokia.com>
Mon, 5 Aug 2019 11:01:58 +0000 (14:01 +0300)
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Wed, 14 Aug 2019 16:22:09 +0000 (18:22 +0200)
commite57cd3fb886d3f298ac3756dcd96d4b142af06a0
tree6d296d29f5634314a0719d0c3dda8194f3d06c07
parent15e19aa1fc388485a5b4c33417189bff4dc3d883
update-utmp: fix assertion failure if rescue.target, multi-user.target and graphical.target are all inactive

If rescue.target, multi-user.target and graphical.target are all
inactive, get_current_runlevel() is not able to determine current
runlevel, and returns with zero. This zero runlevel value results to
assertion failure in utmp_put_runlevel().

 # systemctl stop rescue.target multi-user.target graphical.target
 # systemctl start systemd-update-utmp-runlevel.service

 systemd[1]: Stopped target Graphical Interface.
 systemd[1]: Stopped target Multi-User System.
 systemd[1]: Starting Update UTMP about System Runlevel Changes...
 systemd-update-utmp[67]: Assertion 'runlevel > 0' failed at src/shared/utmp-wtmp.c:275, function utmp_put_runlevel(). Aborting.
 systemd[1]: systemd-update-utmp-runlevel.service: Main process exited, code=dumped, status=6/ABRT
 systemd[1]: systemd-update-utmp-runlevel.service: Failed with result 'core-dump'.
 systemd[1]: Failed to start Update UTMP about System Runlevel Changes.

Let's just print a warning in this case and skip the utmp update, to
avoid systemd-update-utmp-runlevel.service failures.
src/update-utmp/update-utmp.c