summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFrederic Peters <fpeters@entrouvert.com>2005-01-21 21:56:34 +0000
committerFrederic Peters <fpeters@entrouvert.com>2005-01-21 21:56:34 +0000
commitd8d144f7d0d96c437a81ef52b015345ab27a5084 (patch)
tree3a4f4db719e0ead616818a57501134e44424c9fb
parent548c81feb3e794c947b70b8a0b3cba75705a7fe9 (diff)
downloadlasso-d8d144f7d0d96c437a81ef52b015345ab27a5084.tar.gz
lasso-d8d144f7d0d96c437a81ef52b015345ab27a5084.tar.xz
lasso-d8d144f7d0d96c437a81ef52b015345ab27a5084.zip
don't segfault on missing remote provider id
-rw-r--r--lasso/id-ff/login.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/lasso/id-ff/login.c b/lasso/id-ff/login.c
index 81c7eaa1..e87eeda0 100644
--- a/lasso/id-ff/login.c
+++ b/lasso/id-ff/login.c
@@ -468,6 +468,9 @@ lasso_login_build_artifact_msg(LassoLogin *login, LassoHttpMethod http_method)
/* build artifact infos */
remote_provider = g_hash_table_lookup(profile->server->providers,
profile->remote_providerID);
+ if (remote_provider == NULL)
+ return critical_error(LASSO_PROFILE_ERROR_MISSING_REMOTE_PROVIDERID);
+
url = lasso_provider_get_assertion_consumer_service_url(remote_provider,
LASSO_LIB_AUTHN_REQUEST(profile->request)->AssertionConsumerServiceID);
if (url == NULL) {