diff options
author | Ray Strode <rstrode@redhat.com> | 2014-04-10 09:27:59 -0400 |
---|---|---|
committer | Ray Strode <rstrode@redhat.com> | 2014-04-10 13:23:22 -0400 |
commit | 243ede3395e2745254dcef913e0b4bc97af37be5 (patch) | |
tree | 3ba027f6994cfb205ed1d3346a0fd4bea74ca69d | |
parent | c7553e88567133e03804255e0002b3f96fc5aa87 (diff) | |
download | gdm-243ede3395e2745254dcef913e0b4bc97af37be5.tar.gz |
manager: be more accepting of callers with uids different from their session
If a user runs su in their session, that switched user will now be
running in a session that doesn't belong to it. GDM won't allow a user
access to the worker process associated with the session in this case.
Instead, it will try to create a temporary just-in-time reauthentication
channel so reauthentication can happen without having the user talking to
another user's worker. Unfortunately, a logic error in the code means,
the user won't access to its own just-in-time channel.
This commit fixes that.
https://bugzilla.gnome.org/show_bug.cgi?id=727982
-rw-r--r-- | daemon/gdm-manager.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/daemon/gdm-manager.c b/daemon/gdm-manager.c index 97239325..09853d6e 100644 --- a/daemon/gdm-manager.c +++ b/daemon/gdm-manager.c @@ -814,7 +814,7 @@ get_display_and_details_for_bus_sender (GdmManager *self, } if (out_uid != NULL) { - *out_uid = session_uid; + *out_uid = caller_uid; } if (caller_uid != session_uid) { |