diff options
author | Ray Strode <rstrode@redhat.com> | 2010-09-13 10:19:13 -0400 |
---|---|---|
committer | Ray Strode <rstrode@redhat.com> | 2010-09-13 10:19:13 -0400 |
commit | 9027a2cb3db2bbf603540b97ee1398a90053a026 (patch) | |
tree | 952147bda7d393bbb33049dd490047616b8cb343 | |
parent | c690a9b51dca5700bb5e512e6f2e55d9440a145a (diff) | |
download | gdm-9027a2cb3db2bbf603540b97ee1398a90053a026.tar.gz |
Don't call update_item_for_user on non-users in userlist
Previously, get_user() it would fail right away and we
would avoid the call. Now get_user() returns a unloaded
stub object so we need to avoid calling get_user() in the
first place.
-rw-r--r-- | gui/simple-greeter/gdm-user-chooser-widget.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/gui/simple-greeter/gdm-user-chooser-widget.c b/gui/simple-greeter/gdm-user-chooser-widget.c index 211b0331..5aa2cfaf 100644 --- a/gui/simple-greeter/gdm-user-chooser-widget.c +++ b/gui/simple-greeter/gdm-user-chooser-widget.c @@ -235,6 +235,14 @@ on_item_load (GdmChooserWidget *widget, return; } + if (strcmp (id, GDM_USER_CHOOSER_USER_OTHER) == 0) { + return; + } + + if (strcmp (id, GDM_USER_CHOOSER_USER_GUEST) == 0) { + return; + } + user = gdm_user_manager_get_user (user_chooser->priv->manager, id); if (user != NULL) { update_item_for_user (user_chooser, user); |