summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFrederic Peters <fpeters@entrouvert.com>2006-10-29 18:17:00 +0000
committerFrederic Peters <fpeters@entrouvert.com>2006-10-29 18:17:00 +0000
commitdd76935c233ab2f251044bb4b2840e1a8859008e (patch)
treeb63e054bf790d1942e14593e4377f6c2a4d9fd9b
parent803d916410efe24a04028cf22f01a21c3ce9eb24 (diff)
downloadlasso-dd76935c233ab2f251044bb4b2840e1a8859008e.tar.gz
lasso-dd76935c233ab2f251044bb4b2840e1a8859008e.tar.xz
lasso-dd76935c233ab2f251044bb4b2840e1a8859008e.zip
use new error code for missing artifact
-rw-r--r--lasso/id-ff/login.c3
-rw-r--r--lasso/saml-2.0/profile.c5
2 files changed, 7 insertions, 1 deletions
diff --git a/lasso/id-ff/login.c b/lasso/id-ff/login.c
index 2a5dd94c..98f4d4ea 100644
--- a/lasso/id-ff/login.c
+++ b/lasso/id-ff/login.c
@@ -1267,6 +1267,9 @@ lasso_login_init_request(LassoLogin *login, gchar *response_msg,
xmlFree(query_fields[i]);
}
g_free(query_fields);
+ if (artifact_b64 == NULL) {
+ return LASSO_PROFILE_ERROR_MISSING_ARTIFACT;
+ }
}
if (response_http_method == LASSO_HTTP_METHOD_POST) {
artifact_b64 = g_strdup(response_msg);
diff --git a/lasso/saml-2.0/profile.c b/lasso/saml-2.0/profile.c
index 24d88b86..57cc76b6 100644
--- a/lasso/saml-2.0/profile.c
+++ b/lasso/saml-2.0/profile.c
@@ -140,7 +140,7 @@ lasso_saml20_profile_init_artifact_resolve(LassoProfile *profile,
if (method == LASSO_HTTP_METHOD_ARTIFACT_GET) {
query_fields = urlencoded_to_strings(msg);
for (i=0; query_fields[i]; i++) {
- if (strncmp(query_fields[i], "SAMLArt=", 8) != 0) {
+ if (strncmp(query_fields[i], "SAMLart=", 8) != 0) {
xmlFree(query_fields[i]);
continue;
}
@@ -148,6 +148,9 @@ lasso_saml20_profile_init_artifact_resolve(LassoProfile *profile,
xmlFree(query_fields[i]);
}
g_free(query_fields);
+ if (artifact_b64 == NULL) {
+ return LASSO_PROFILE_ERROR_MISSING_ARTIFACT;
+ }
} else {
artifact_b64 = g_strdup(msg);
}