]> git.ipfire.org Git - thirdparty/plymouth.git/commit
ply-device-manager: Don't deactivate renderers from ply_device_manager_free ()
authorHans de Goede <hdegoede@redhat.com>
Tue, 30 Mar 2021 18:07:53 +0000 (20:07 +0200)
committerHans de Goede <hdegoede@redhat.com>
Wed, 31 Mar 2021 14:35:27 +0000 (16:35 +0200)
commit630ca2e18fc63c3e144f11b54faf70228d4d2e92
treed86e26cb53687a79e823b392e186144286d6cf92
parentfbe2b79da56cf0a4b7ed5ca4754449ab4875b40f
ply-device-manager: Don't deactivate renderers from ply_device_manager_free ()

Don't deactivate renderers from ply_device_manager_free (),
ply_device_manager_free () is only ever called with the renderers still
active on a "plymouth quit --retain-splash".

Since the splash is being retained in this case the renderers should not be
deactivated. Normally this does not matter because plymouthd exits almost
immediately afterwards and the kernels will close the renderers fds on exit.

But with the upcoming plymouthd-drm-escrow binary which keeps the renderers
fds open, not deactivate renderers; and thus not dropping DRM master rights
does make a difference.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
src/libply-splash-core/ply-device-manager.c