diff options
| author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2009-04-30 14:58:16 +0000 |
|---|---|---|
| committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2009-04-30 14:58:16 +0000 |
| commit | 7e6dfb76834f4ed5ebe574572559004e95d2bdea (patch) | |
| tree | 7c171177e47a5ac70addd5f00ad550ef5e1db65b /tests/random_tests.c | |
| parent | e863eefdeb0fcd692958d33c860baa53bda2f202 (diff) | |
| download | lasso-7e6dfb76834f4ed5ebe574572559004e95d2bdea.tar.gz lasso-7e6dfb76834f4ed5ebe574572559004e95d2bdea.tar.xz lasso-7e6dfb76834f4ed5ebe574572559004e95d2bdea.zip | |
Fix leaks in tests
* basic_tests.c:
* login_tests.c:
* login_tests_saml2.c:
* random_tests.c:
free replaced string, unref used nodes, initialize local variables
when necessary, free newly allocated strings.
Diffstat (limited to 'tests/random_tests.c')
| -rw-r--r-- | tests/random_tests.c | 20 |
1 files changed, 15 insertions, 5 deletions
diff --git a/tests/random_tests.c b/tests/random_tests.c index 47b5aaee..33841c17 100644 --- a/tests/random_tests.c +++ b/tests/random_tests.c @@ -32,6 +32,7 @@ #include <lasso/xml/lib_authentication_statement.h> #include <lasso/xml/saml_name_identifier.h> #include <lasso/xml/samlp_response.h> +#include "../lasso/utils.h" Suite* random_suite(); @@ -49,6 +50,7 @@ START_TEST(test01_provider_new) dump = lasso_node_dump(LASSO_NODE(provider)); printf("dump:\n%s\n", dump); g_object_unref(provider); + lasso_release_string(dump); } END_TEST @@ -63,12 +65,13 @@ START_TEST(test02_provider_new_from_dump) TESTSDATADIR "/ca1-la/certificate.pem"); dump = lasso_node_dump(LASSO_NODE(provider1)); - provider2 = lasso_provider_new_from_dump(dump); + lasso_release_string(dump); dump = lasso_node_dump(LASSO_NODE(provider2)); printf("dump:\n%s\n", dump); g_object_unref(provider1); g_object_unref(provider2); + lasso_release_string(dump); } END_TEST @@ -86,6 +89,7 @@ START_TEST(test01_server_new) dump = lasso_node_dump(LASSO_NODE(server)); printf("dump:%s\n", dump); g_object_unref(server); + lasso_release_string(dump); } END_TEST @@ -108,6 +112,7 @@ START_TEST(test02_server_add_provider) dump = lasso_node_dump(LASSO_NODE(server)); g_object_unref(server); + lasso_release_string(dump); } END_TEST @@ -205,17 +210,22 @@ START_TEST(test06_lib_statuscode) * starts with lib:, that namespace must be defined. (was bug#416) */ LassoSamlpResponse *response = LASSO_SAMLP_RESPONSE(lasso_samlp_response_new()); + char *dump = NULL; - response->Status->StatusCode->Value = g_strdup(LASSO_SAML_STATUS_CODE_SUCCESS); - fail_unless(strstr(lasso_node_dump(LASSO_NODE(response)), "xmlns:lib=") == NULL, + lasso_assign_string(response->Status->StatusCode->Value, LASSO_SAML_STATUS_CODE_SUCCESS); + dump = lasso_node_dump(LASSO_NODE(response)); + fail_unless(strstr(dump, "xmlns:lib=") == NULL, "liberty namespace should not be defined"); + lasso_release_string(dump); - response->Status->StatusCode->Value = g_strdup(LASSO_SAML_STATUS_CODE_RESPONDER); + lasso_assign_string(response->Status->StatusCode->Value, LASSO_SAML_STATUS_CODE_RESPONDER); response->Status->StatusCode->StatusCode = lasso_samlp_status_code_new(); response->Status->StatusCode->StatusCode->Value = g_strdup( LASSO_LIB_STATUS_CODE_UNKNOWN_PRINCIPAL); - fail_unless(strstr(lasso_node_dump(LASSO_NODE(response)), "xmlns:lib=") != NULL, + dump = lasso_node_dump(LASSO_NODE(response)); + fail_unless(strstr(dump, "xmlns:lib=") != NULL, "liberty namespace should be defined"); + lasso_release_string(dump); g_object_unref(response); } END_TEST |