← Back to team overview

unity-api-bugs team mailing list archive

[Bug 1263438] Re: Indicator-session assumes there's only one active user

 

I tested the getuid() change and it works fine on LTSP thin and fat
clients.

-- 
You received this bug notification because you are a member of Unity API
bugs, which is subscribed to Session Menu.
https://bugs.launchpad.net/bugs/1263438

Title:
  Indicator-session assumes there's only one active user

Status in The Session Menu:
  New

Bug description:
  In the policykit sense, an "active" user is one that is using a physical console, and so has rights to shutdown the system etc.
  Remote users, like NX users, LTSP users, XDMCP users, or users that use X+VNC scripts to remotely log in, are correctly not considered active, and thus don't have some privileges like shutting down the system.

  The problem is that indicator-session is using the policykit meaning of the word "active",
  instead of checking for "currently logged in and working" users,
  i.e. it thinks that only one user can be working on a PC at a given time.

  From src/backend-dbus/users.c, line 593:
        ret->is_current_user = uid == p->active_uid;

  So if I'm logged in from a thin client, or via NX, or via XDMCP etc etc, indicator-session doesn't match the uid of its own process to tell me who I am, but just relies on the matching of the "active" user.
  Thus I can't see my username if I enable "show-real-name-on-panel", I don't see the dot that means "current user" in the menu's user list etc.

  A solution would be to use getuid() in the code above.

To manage notifications about this bug go to:
https://bugs.launchpad.net/indicator-session/+bug/1263438/+subscriptions


References