diff options
| author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2009-10-30 14:47:24 +0000 |
|---|---|---|
| committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2009-10-30 14:47:24 +0000 |
| commit | 9eb77cf9fe825b78aee1564cba994b2b4e1196f7 (patch) | |
| tree | 35481385d92b9e1c3db2e7b62025695f7c255e2a | |
| parent | 6921dd1d9b6e398edcfcdab991e61d161457b7c0 (diff) | |
| download | lasso-9eb77cf9fe825b78aee1564cba994b2b4e1196f7.tar.gz lasso-9eb77cf9fe825b78aee1564cba994b2b4e1196f7.tar.xz lasso-9eb77cf9fe825b78aee1564cba994b2b4e1196f7.zip | |
SAML2: in logout_build_response_msg do not fail on missing remote_providerID
* lasso/saml-2.0/logout.c:
remote_providerId is verified in build_redirect_simple and is not
necessary for SOAP response with a failure and profile->response
cannot be null (we just created a response if it was missing).
So this error case is now unnecessary.
| -rw-r--r-- | lasso/saml-2.0/logout.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/lasso/saml-2.0/logout.c b/lasso/saml-2.0/logout.c index 599a99d4..083d56fe 100644 --- a/lasso/saml-2.0/logout.c +++ b/lasso/saml-2.0/logout.c @@ -459,13 +459,6 @@ lasso_saml20_logout_build_response_msg(LassoLogout *logout) lasso_assign_string(response->certificate_file, profile->server->certificate); } - if (profile->remote_providerID == NULL || profile->response == NULL) { - /* no remote provider id set or no response set, this means - * this function got called before validate_request, probably - * because there were no active session */ - return critical_error(LASSO_SERVER_ERROR_PROVIDER_NOT_FOUND); - } - /* build logout response message */ if (profile->http_request_method == LASSO_HTTP_METHOD_SOAP) { lasso_release_string(profile->msg_url); |