summaryrefslogtreecommitdiffstats
path: root/lasso
diff options
context:
space:
mode:
authorBenjamin Dauvergne <bdauvergne@entrouvert.com>2010-12-21 16:38:57 +0100
committerBenjamin Dauvergne <bdauvergne@entrouvert.com>2010-12-21 16:38:57 +0100
commitcfd58003fa315193b512aeb8b2a78431b35100dd (patch)
tree2bece5507bfb714dcda5604e3722b3679b4f0581 /lasso
parente401253f8def3bd0cd3fa21bf0683fb999cc6d65 (diff)
downloadlasso-cfd58003fa315193b512aeb8b2a78431b35100dd.tar.gz
lasso-cfd58003fa315193b512aeb8b2a78431b35100dd.tar.xz
lasso-cfd58003fa315193b512aeb8b2a78431b35100dd.zip
[saml2] fix errors in lasso)provider_get_first_http_method when a binding is unknown
Diffstat (limited to 'lasso')
-rw-r--r--lasso/saml-2.0/provider.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/lasso/saml-2.0/provider.c b/lasso/saml-2.0/provider.c
index 431f2937..4f9192d5 100644
--- a/lasso/saml-2.0/provider.c
+++ b/lasso/saml-2.0/provider.c
@@ -518,7 +518,6 @@ lasso_saml20_provider_get_first_http_method(G_GNUC_UNUSED LassoProvider *provide
kind = profile_names[protocol_type];
}
if (! kind) {
- warning("Could not find a first http method for protocol type %u", protocol_type);
return LASSO_HTTP_METHOD_NONE;
}
@@ -526,7 +525,8 @@ lasso_saml20_provider_get_first_http_method(G_GNUC_UNUSED LassoProvider *provide
EndpointType *endpoint_type = (EndpointType*)t->data;
if (endpoint_type && lasso_strisequal(endpoint_type->kind, kind)) {
result = binding_uri_to_http_method(endpoint_type->binding);
- if (result) break;
+ if (result != LASSO_HTTP_METHOD_NONE)
+ break;
}
}
@@ -546,7 +546,7 @@ lasso_saml20_provider_accept_http_method(G_GNUC_UNUSED LassoProvider *provider,
}
if (! kind) {
warning("Could not find a first http method for protocol type %u", protocol_type);
- return LASSO_HTTP_METHOD_NONE;
+ return FALSE;
}
lasso_foreach(t, remote_provider->private_data->endpoints) {