diff options
| author | Ezra Peisach <epeisach@mit.edu> | 2009-07-30 17:22:28 +0000 |
|---|---|---|
| committer | Ezra Peisach <epeisach@mit.edu> | 2009-07-30 17:22:28 +0000 |
| commit | 4ab2266afad7a3ede0fa3587d7a38e855948af55 (patch) | |
| tree | dd060ee36807ae59d0ff3369d6028820a5f3255e /src | |
| parent | 6b98b9f4f7f2e16d8e48081544d857f15bfbf5d0 (diff) | |
| download | krb5-4ab2266afad7a3ede0fa3587d7a38e855948af55.tar.gz krb5-4ab2266afad7a3ede0fa3587d7a38e855948af55.tar.xz krb5-4ab2266afad7a3ede0fa3587d7a38e855948af55.zip | |
Fix memory leak in k5_pac_verify_server_checksum
k5_pac_verify_server_checksum was leaking memory when the checksum was valid.
t_pac.c: Fix memory leak by forgetting to release memory.
ticket: 6541
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@22474 dc483132-0cff-0310-8789-dd5450dbe970
Diffstat (limited to 'src')
| -rw-r--r-- | src/lib/krb5/krb/pac.c | 4 | ||||
| -rw-r--r-- | src/lib/krb5/krb/t_pac.c | 2 |
2 files changed, 5 insertions, 1 deletions
diff --git a/src/lib/krb5/krb/pac.c b/src/lib/krb5/krb/pac.c index 30a63bf35..c5a706562 100644 --- a/src/lib/krb5/krb/pac.c +++ b/src/lib/krb5/krb/pac.c @@ -547,8 +547,10 @@ k5_pac_verify_server_checksum(krb5_context context, ret = krb5_c_verify_checksum(context, server, KRB5_KEYUSAGE_APP_DATA_CKSUM, &pac_data, &checksum, &valid); + + free(pac_data.data); + if (ret != 0) { - free(pac_data.data); return ret; } diff --git a/src/lib/krb5/krb/t_pac.c b/src/lib/krb5/krb/t_pac.c index 527e3db22..503d778a9 100644 --- a/src/lib/krb5/krb/t_pac.c +++ b/src/lib/krb5/krb/t_pac.c @@ -220,6 +220,8 @@ main(int argc, char **argv) if (ret) err(context, ret, "krb5_pac_verify 4"); + krb5_free_data_contents(context, &data); + krb5_pac_free(context, pac2); } |