Try to not leak memory.

git-svn-id: svn://svn.h5l.se/heimdal/trunk/heimdal@18809 ec53bebd-3082-4978-b11e-865c3cabbd6b
This commit is contained in:
Love Hörnquist Åstrand
2006-10-22 07:11:43 +00:00
parent 09196ad8d5
commit 8ea293e426
6 changed files with 23 additions and 23 deletions

View File

@@ -337,6 +337,7 @@ test_cache_find(krb5_context context, const char *type, const char *principal,
if (id)
krb5_cc_close(context, id);
krb5_free_principal(context, client);
}

View File

@@ -89,6 +89,7 @@ test_wrapping(krb5_context context,
krb5_data_free(&data);
}
free(etype_name);
free(buf);
krb5_crypto_destroy(context, crypto);
krb5_free_keyblock_contents(context, &key);

View File

@@ -163,7 +163,8 @@ test_memory_keytab(krb5_context context, const char *keytab, const char *keytab2
if (ret)
krb5_err(context, 1, ret, "krb5_kt_close");
krb5_free_principal(context, entry3.principal);
krb5_free_keyblock_contents(context, &entry3.keyblock);
}
int

View File

@@ -60,6 +60,8 @@ test_dh2key(int i,
if (key.keyvalue.length != result->length ||
memcmp(key.keyvalue.data, result->data, result->length) != 0)
krb5_errx(context, 1, "resulting key wrong: %d", i);
krb5_free_keyblock_contents(context, &key);
}

View File

@@ -72,15 +72,16 @@ test_princ(krb5_context context)
if (ret)
krb5_err(context, 1, ret, "krb5_parse_name");
if (strcmp(princ_short, princ_unparsed)) {
if (strcmp(princ_short, princ_unparsed))
krb5_errx(context, 1, "%s != %s", princ_short, princ_unparsed);
}
free(princ_unparsed);
realm = krb5_principal_get_realm(context, p);
asprintf(&princ_reformed, "%s@%s", princ_short, realm);
ret = krb5_parse_name(context, princ_reformed, &p2);
free(princ_reformed);
if (ret)
krb5_err(context, 1, ret, "krb5_parse_name");
@@ -100,27 +101,25 @@ test_princ(krb5_context context)
if (ret)
krb5_err(context, 1, ret, "krb5_parse_name");
if (strcmp(princ_short, princ_unparsed)) {
if (strcmp(princ_short, princ_unparsed))
krb5_errx(context, 1, "'%s' != '%s'", princ_short, princ_unparsed);
}
free(princ_unparsed);
ret = krb5_parse_name(context, princ_short, &p2);
if (ret)
krb5_err(context, 1, ret, "krb5_parse_name");
if (!krb5_principal_compare(context, p, p2)) {
if (!krb5_principal_compare(context, p, p2))
krb5_errx(context, 1, "p != p2");
}
krb5_free_principal(context, p2);
ret = krb5_unparse_name(context, p, &princ_unparsed);
if (ret)
krb5_err(context, 1, ret, "krb5_parse_name");
if (strcmp(princ, princ_unparsed)) {
if (strcmp(princ, princ_unparsed))
krb5_errx(context, 1, "'%s' != '%s'", princ, princ_unparsed);
}
free(princ_unparsed);
ret = krb5_set_default_realm(context, "SAMBA.ORG");
if (ret)
@@ -130,21 +129,19 @@ test_princ(krb5_context context)
if (ret)
krb5_err(context, 1, ret, "krb5_parse_name");
if (krb5_principal_compare(context, p, p2)) {
if (krb5_principal_compare(context, p, p2))
krb5_errx(context, 1, "p == p2");
}
if (!krb5_principal_compare_any_realm(context, p, p2)) {
if (!krb5_principal_compare_any_realm(context, p, p2))
krb5_errx(context, 1, "(ignoring realms) p != p2");
}
ret = krb5_unparse_name(context, p2, &princ_unparsed);
if (ret)
krb5_err(context, 1, ret, "krb5_parse_name");
if (strcmp(princ, princ_unparsed) == 0) {
if (strcmp(princ, princ_unparsed) == 0)
krb5_errx(context, 1, "%s == %s", princ, princ_unparsed);
}
free(princ_unparsed);
krb5_free_principal(context, p2);
@@ -159,9 +156,9 @@ test_princ(krb5_context context)
if (ret)
krb5_err(context, 1, ret, "krb5_parse_name");
if (strcmp(princ, princ_unparsed)) {
if (strcmp(princ, princ_unparsed))
krb5_errx(context, 1, "'%s' != '%s'", princ, princ_unparsed);
}
free(princ_unparsed);
krb5_free_principal(context, p2);
@@ -173,7 +170,6 @@ test_princ(krb5_context context)
if (strcmp(princ, princ_unparsed) != 0)
krb5_errx(context, 1, "'%s' != '%s'", princ, princ_unparsed);
free(princ_unparsed);
ret = krb5_unparse_name(context, p, &princ_unparsed);
@@ -182,7 +178,6 @@ test_princ(krb5_context context)
if (strcmp(princ, princ_unparsed))
krb5_errx(context, 1, "'%s' != '%s'", princ, princ_unparsed);
free(princ_unparsed);
ret = krb5_parse_name_flags(context, princ,
@@ -204,9 +199,8 @@ test_princ(krb5_context context)
if (ret)
krb5_err(context, 1, ret, "krb5_unparse_name_norealm");
if (strcmp(princ_short, princ_unparsed)) {
if (strcmp(princ_short, princ_unparsed))
krb5_errx(context, 1, "'%s' != '%s'", princ_short, princ_unparsed);
}
free(princ_unparsed);
ret = krb5_parse_name_flags(context, princ_short,
@@ -233,7 +227,6 @@ test_princ(krb5_context context)
if (strcmp(princ_short, princ_unparsed))
krb5_errx(context, 1, "'%s' != '%s'", princ_short, princ_unparsed);
free(princ_unparsed);
krb5_free_principal(context, p);

View File

@@ -81,5 +81,7 @@ main(int argc, char **argv)
check_set_time(context);
check_set_time(context);
krb5_free_context(context);
return 0;
}