diff options
| author | Nicolas Clapies <nclapies@entrouvert.com> | 2004-08-30 15:36:19 +0000 |
|---|---|---|
| committer | Nicolas Clapies <nclapies@entrouvert.com> | 2004-08-30 15:36:19 +0000 |
| commit | 289c08c485492bdbdbf3610f004838c2b3cc0744 (patch) | |
| tree | 707ebe23bf2ed81c8757a18a33b2a3aa5d4c685d | |
| parent | 5372db6d37c14951430f80d9361fff66a0ee8385 (diff) | |
| download | lasso-289c08c485492bdbdbf3610f004838c2b3cc0744.tar.gz lasso-289c08c485492bdbdbf3610f004838c2b3cc0744.tar.xz lasso-289c08c485492bdbdbf3610f004838c2b3cc0744.zip | |
Added a test in lasso_defederation_build_notification_msg() to test if the federation termination service url exists
| -rw-r--r-- | lasso/id-ff/defederation.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/lasso/id-ff/defederation.c b/lasso/id-ff/defederation.c index 36ea809e..8fbf2e40 100644 --- a/lasso/id-ff/defederation.c +++ b/lasso/id-ff/defederation.c @@ -128,16 +128,17 @@ lasso_defederation_build_notification_msg(LassoDefederation *defederation) query = lasso_node_export_to_query(profile->request, profile->server->signature_method, profile->server->private_key); - profile->msg_url = g_new(gchar, strlen(url)+strlen(query)+1+1); - g_sprintf(profile->msg_url, "%s?%s", url, query); - profile->msg_body = NULL; - if (profile->msg_url == NULL) { - message(G_LOG_LEVEL_CRITICAL, "Error while setting msg_url\n"); + if ( (url == NULL) || (query == NULL) ) { + message(G_LOG_LEVEL_CRITICAL, "%d, Url %s or QUERY %s is NULL\n", remote_provider_type, url, query); ret = -1; goto done; } + profile->msg_url = g_new(gchar, strlen(url)+strlen(query)+1+1); + g_sprintf(profile->msg_url, "%s?%s", url, query); + profile->msg_body = NULL; + xmlFree(url); xmlFree(query); } |