[systemd-devel] [PATCH] logind: handle closing sessions over daemon restarts

[Lennart Poettering]

On Wed, 28.01.15 18:42, Martin Pitt (martin.pitt at ubuntu.com) wrote:

> Hello,
> 
> We restart logind in our systemd package on package upgrades, as that
> should generally be safe and our policy is to do that whenever
> possible to avoid reboots and apply security and other fixes
> immediately.
> 
> We got several reports (https://launchpad.net/bugs/1415104) about
> users losing their device ACLs and polkit privileges when they install
> package updates involving systemd. I found out that this happens if
> you have older sessions around in state "closing", i. e. there's still
> some leftover process in them [1]. After restarting logind, that old
> "closing" session suddenly becomes Active=yes, and the "real" active
> session becomes Active=no.
> 
> Patch attached, I tested it fairly thoroughly.
> 
> Thanks for considering!

Looks good! Applied! Thanks!

Lennart

-- 
Lennart Poettering, Red Hat