Comment 6 for bug 199486

Revision history for this message
In , Michael Frey (mfrey) wrote :

(In reply to comment #0)
> (not sure I've filed this against the right component, please advise)
>
> ConsoleKit [1] is used to track user desktop sessions so to work with
> ConsoleKit any process used to launch these needs to poke ConsoleKit so the
> session can be registered. Will attach a patch to xinit.c that does this.
>
> [1] :
> http://gitweb.freedesktop.org/?p=ConsoleKit.git
> http://people.freedesktop.org/~david/ConsoleKit.html (a bit outdated)
>
> Btw, this patch originated from this RH bug
> https://bugzilla.redhat.com/show_bug.cgi?id=233183
>

(In reply to comment #5)
> @jcristau: indeed, makes sense; it will work as is because ck-connector
> requires dbus in the .pc.
>
> Using this patch, it seems that the session isn't made the active one; after
> discussion with other people, one thing which might help is that
> ck_connector_open_session_with_parameters should be used instead of
> ck_connector_open_session with some parameters from:
> display-device
> x11-display-device
> x11-display
>
> (probably session-type, is-local, and unix-user can be set easily as well)
>

Having done more research on this and actually tried to update this patch to use the above mentioned call (open_session_with_parameters), this will not work.

Typically xinit is not run as a root, so the user does not have enough permissions to create a session with parameters.

I question weather or not we should try to move this call into X? X is setuid root.