diff options
| author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2012-09-26 20:52:59 +0200 |
|---|---|---|
| committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2012-09-26 21:01:15 +0200 |
| commit | e94015f8bcc168c9882348d2e8c5a5138ea56676 (patch) | |
| tree | 5bf104448202007f279b58840897bdb2edc153ba /lasso/saml-2.0 | |
| parent | 050afd224676807a8003315e13ba4aee74d7a18a (diff) | |
| download | lasso-e94015f8bcc168c9882348d2e8c5a5138ea56676.tar.gz lasso-e94015f8bcc168c9882348d2e8c5a5138ea56676.tar.xz lasso-e94015f8bcc168c9882348d2e8c5a5138ea56676.zip | |
fix segfault in saml-2.0/provider.c:load_endpoint_type2
Thanks to Hiromitsu Fujita for the patch. fixes #1582.
Diffstat (limited to 'lasso/saml-2.0')
| -rw-r--r-- | lasso/saml-2.0/provider.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lasso/saml-2.0/provider.c b/lasso/saml-2.0/provider.c index 98561d83..a66ba9f2 100644 --- a/lasso/saml-2.0/provider.c +++ b/lasso/saml-2.0/provider.c @@ -180,7 +180,7 @@ load_endpoint_type2(xmlNode *xmlnode, LassoProvider *provider, LassoProviderRole } indexed_endpoint = checkSaml2MdNode(xmlnode, LASSO_SAML2_METADATA_ELEMENT_ASSERTION_CONSUMER_SERVICE); if (indexed_endpoint) { - if (! xsdUnsignedShortParse(index, &idx)) { + if (! index || ! xsdUnsignedShortParse(index, &idx)) { warning("Invalid AssertionConsumerService, no index set"); goto cleanup; } |