applications/gnome-control-center
4
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

user-accounts: check for NULL on um_realm_object_get_common

Browse source 
um_realm_object_get_common can return NULL, which shouldn't be passed
into subsequent functions to avoid crashes.

https://bugzilla.gnome.org/show_bug.cgi?id=705351
This commit is contained in:
Ondrej Holy 2015-02-25 14:31:28 +01:00
parent 15a1bf7196
commit 3e6b741ad2
3 changed files with 17 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
panels/user-accounts/um-account-dialog.c
View file

@ -655,6 +655,8 @@ enterprise_add_realm (UmAccountDialog *self,
gchar *name;
common = um_realm_object_get_common (realm);
g_return_if_fail (common != NULL);
realm_name = um_realm_common_get_name (common);
/*
@ -793,6 +795,12 @@ enterprise_permit_user_login (UmAccountDialog *self)
GVariant *options;
common = um_realm_object_get_common (self->selected_realm);
if (common != NULL) {
g_debug ("Failed to register account: failed to get d-bus interface");
show_error_dialog (self, _("Failed to register account"), NULL);
finish_action (self);
return;
}
login = um_realm_calculate_login (common, gtk_entry_get_text (self->enterprise_login));
g_return_if_fail (login != NULL);

10
panels/user-accounts/um-realm-manager.c
View file

@ -479,12 +479,14 @@ um_realm_is_configured (UmRealmObject *realm)
{
UmRealmCommon *common;
const gchar *configured;
gboolean is;
gboolean is = FALSE;
common = um_realm_object_get_common (realm);
configured = um_realm_common_get_configured (common);
is = configured != NULL && !g_str_equal (configured, "");
g_object_unref (common);
if (common != NULL) {
configured = um_realm_common_get_configured (common);
is = configured != NULL && !g_str_equal (configured, "");
g_object_unref (common);
}
return is;
}

3
panels/user-accounts/um-user-panel.c
View file

@ -524,6 +524,9 @@ find_matching_realm (UmRealmManager *realm_manager, const gchar *login)
gint i;
common = um_realm_object_get_common (l->data);
if (common == NULL)
continue;
permitted_logins = um_realm_common_get_permitted_logins (common);
for (i = 0; permitted_logins[i] != NULL; i++) {
if (g_strcmp0 (permitted_logins[i], login) == 0)