[PATCH v2 10/14] systemd-logind: Add systemd-logind "core"

[David Herrmann]

Hi

On Mon, Feb 10, 2014 at 10:48 PM, Hans de Goede <hdegoede at redhat.com> wrote:
> Hi,
>
> On 02/10/2014 09:58 PM, Jasper St. Pierre wrote:
>> Something I noticed here is that you use GetSessionByPID(). This works
>> right now, but with systemd user sessions, the display server will run
>> outside of a session. We have to decide what to do in this case.
>
> AFAIK that won't work, the display server must be inside the pam login
> session of the user, otherwise logind will refuse to give access to any
> of the devices belonging to the seat.
>
> To be precise, logind will only give access to a process which is
> inside the user-session which is the active session on a certain head,
> and then only to devices which belong to said head.

To be precise, only processes with the uid of the user or root can
call TakeControl(), no other restrictions are enforced. So the process
does not have to be in the session.

Cheers
David