From f9bb1b81fed053991324de84d6856ee61188aa0f Mon Sep 17 00:00:00 2001 From: Lukas Slebodnik Date: Mon, 14 Oct 2013 11:08:48 +0200 Subject: krb5: Use right function to free data. In function create_empty_cred, krb5_creds was aloocated using calloc, but krb5_free_creds was used to remove this creds in done section. Therefore clang static analyzer repoted this as warning: Potential leak of memory pointed to by 'cred' --- src/providers/krb5/krb5_child.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/providers/krb5/krb5_child.c b/src/providers/krb5/krb5_child.c index f96bc8aae..bd937e808 100644 --- a/src/providers/krb5/krb5_child.c +++ b/src/providers/krb5/krb5_child.c @@ -424,7 +424,8 @@ static krb5_error_code create_empty_cred(krb5_context ctx, krb5_principal princ, done: if (kerr != 0) { - krb5_free_creds(ctx, cred); + krb5_free_cred_contents(ctx, cred); + free(cred); } else { *_cred = cred; } -- cgit