|
 |
db363b |
From 20f653f70818b85fe1b4de77a629fce352fb8cbd Mon Sep 17 00:00:00 2001
|
|
|
db363b |
From: Jakub Hrozek <jhrozek@redhat.com>
|
|
|
db363b |
Date: Mon, 26 Jul 2021 16:25:52 +0200
|
|
|
db363b |
Subject: [PATCH] lasso_saml20_login_process_response_status_and_assertion:
|
|
|
db363b |
handle rc as per verify_hint
|
|
|
db363b |
|
|
|
db363b |
In case VERIFY_HINT was set to IGNORE and the login signature was
|
|
|
db363b |
incorrect, lasso_saml20_login_process_response_status_and_assertion
|
|
|
db363b |
would have jumped straight to the cleanup label which just returns the
|
|
|
db363b |
return code. Let's jump to a new label handlerc instead which might set
|
|
|
db363b |
the return code to 0 in case verify_hint is set to IGNORE.
|
|
|
db363b |
|
|
|
db363b |
Related: https://dev.entrouvert.org/issues/54689
|
|
|
db363b |
---
|
|
|
db363b |
lasso/saml-2.0/login.c | 20 ++++++--------------
|
|
|
db363b |
1 file changed, 6 insertions(+), 14 deletions(-)
|
|
|
db363b |
|
|
|
db363b |
diff --git a/lasso/saml-2.0/login.c b/lasso/saml-2.0/login.c
|
|
|
db363b |
index cf62c1cc9..1d5668b5b 100644
|
|
|
db363b |
--- a/lasso/saml-2.0/login.c
|
|
|
db363b |
+++ b/lasso/saml-2.0/login.c
|
|
|
db363b |
@@ -1371,7 +1371,7 @@ lasso_saml20_login_process_response_status_and_assertion(LassoLogin *login)
|
|
|
db363b |
char *status_value;
|
|
|
db363b |
lasso_error_t rc = 0;
|
|
|
db363b |
lasso_error_t assertion_signature_status = 0;
|
|
|
db363b |
- LassoProfileSignatureVerifyHint verify_hint;
|
|
|
db363b |
+ LassoProfileSignatureVerifyHint verify_hint = LASSO_PROFILE_SIGNATURE_VERIFY_HINT_LAST;
|
|
|
db363b |
|
|
|
db363b |
profile = &login->parent;
|
|
|
db363b |
lasso_extract_node_or_fail(response, profile->response, SAMLP2_STATUS_RESPONSE,
|
|
|
db363b |
@@ -1492,20 +1492,12 @@ lasso_saml20_login_process_response_status_and_assertion(LassoLogin *login)
|
|
|
db363b |
lasso_assign_gobject (login->private_data->saml2_assertion, last_assertion);
|
|
|
db363b |
}
|
|
|
db363b |
|
|
|
db363b |
- switch (verify_hint) {
|
|
|
db363b |
- case LASSO_PROFILE_SIGNATURE_VERIFY_HINT_FORCE:
|
|
|
db363b |
- case LASSO_PROFILE_SIGNATURE_VERIFY_HINT_MAYBE:
|
|
|
db363b |
- break;
|
|
|
db363b |
- case LASSO_PROFILE_SIGNATURE_VERIFY_HINT_IGNORE:
|
|
|
db363b |
- /* ignore signature errors */
|
|
|
db363b |
- if (rc == LASSO_PROFILE_ERROR_CANNOT_VERIFY_SIGNATURE) {
|
|
|
db363b |
- rc = 0;
|
|
|
db363b |
- }
|
|
|
db363b |
- break;
|
|
|
db363b |
- default:
|
|
|
db363b |
- g_assert(0);
|
|
|
db363b |
- }
|
|
|
db363b |
cleanup:
|
|
|
db363b |
+ if (verify_hint == LASSO_PROFILE_SIGNATURE_VERIFY_HINT_IGNORE &&
|
|
|
db363b |
+ rc == LASSO_PROFILE_ERROR_CANNOT_VERIFY_SIGNATURE) {
|
|
|
db363b |
+ profile->signature_status = rc;
|
|
|
db363b |
+ rc = 0;
|
|
|
db363b |
+ }
|
|
|
db363b |
return rc;
|
|
|
db363b |
}
|
|
|
db363b |
|
|
|
db363b |
--
|
|
|
db363b |
2.26.3
|
|
|
db363b |
|