summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorVivek Dasmohapatra <vivek@collabora.co.uk>2011-03-03 19:50:38 +0000
committerVivek Dasmohapatra <vivek@collabora.co.uk>2011-03-03 20:09:21 +0000
commitbf7751f352477c37047255ba4ae3ed4f90cde8d8 (patch)
tree457c4300141de8ccf7a48b8e94f7f25530aa47df
parent03b1e1d3ebe2a9c48948eef41814e6db6eda45c6 (diff)
downloadtelepathy-mission-control-bf7751f352477c37047255ba4ae3ed4f90cde8d8.tar.gz
It is possible for the AG API to pass us duff account ids. Check for these.
-rw-r--r--src/mcd-account-manager-sso.c7
1 files changed, 7 insertions, 0 deletions
diff --git a/src/mcd-account-manager-sso.c b/src/mcd-account-manager-sso.c
index ce5902a7..e2101aa9 100644
--- a/src/mcd-account-manager-sso.c
+++ b/src/mcd-account-manager-sso.c
@@ -346,6 +346,7 @@ _maybe_set_account_param_from_service (
GValue ag_value = { 0 };
g_return_if_fail (setting != NULL);
+ g_return_if_fail (ag_account != NULL);
g_value_init (&ag_value, G_TYPE_STRING);
@@ -786,6 +787,12 @@ _ag_accountid_to_mc_key (McdAccountManagerSso *sso,
AgSettingSource src = AG_SETTING_SOURCE_NONE;
GValue value = { 0 };
+ if (account == NULL)
+ {
+ DEBUG ("AG Account ID %u invalid", id);
+ return NULL;
+ }
+
DEBUG ("AG Account ID: %u", id);
g_value_init (&value, G_TYPE_STRING);