summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorThomas Oulevey <thomas.oulevey@cern.ch>2015-09-23 10:55:59 +0200
committerJakub Hrozek <jhrozek@redhat.com>2015-10-05 11:07:01 +0200
commitd6073c978b0aac897aee648670fa6b69503ae676 (patch)
tree5ba00cc7ae4e4c473e3a1b9056fed17f9279d9d8
parentf2520325ef26def6680552da58b4cb3abe4523e1 (diff)
downloadsssd-d6073c978b0aac897aee648670fa6b69503ae676.tar.gz
sssd-d6073c978b0aac897aee648670fa6b69503ae676.tar.xz
sssd-d6073c978b0aac897aee648670fa6b69503ae676.zip
Fix memory leak in sssdpac_verify()
Resolves https://fedorahosted.org/sssd/ticket/2803 Reviewed-by: Sumit Bose <sbose@redhat.com> (cherry picked from commit b4c44ebb8997d3debb33607c123ccfd9926e0cba)
-rw-r--r--src/sss_client/sssd_pac.c3
1 files changed, 3 insertions, 0 deletions
diff --git a/src/sss_client/sssd_pac.c b/src/sss_client/sssd_pac.c
index d1790df50..8b5bb3295 100644
--- a/src/sss_client/sssd_pac.c
+++ b/src/sss_client/sssd_pac.c
@@ -150,6 +150,9 @@ static krb5_error_code sssdpac_verify(krb5_context kcontext,
kerr = krb5_pac_verify(kcontext, pac,
req->ticket->enc_part2->times.authtime,
req->ticket->enc_part2->client, key, NULL);
+ /* deallocate pac */
+ krb5_pac_free(kcontext, pac);
+ pac = NULL;
if (kerr != 0) {
/* The krb5 documentation says:
* A checksum mismatch can occur if the PAC was copied from a