diff options
| author | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-12-04 14:24:43 +0100 |
|---|---|---|
| committer | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-12-04 15:14:46 +0100 |
| commit | 3fb5ae600efaeeb1cb7ee02d465b49bc52d111bf (patch) | |
| tree | c3ad2b07295f5323379fe420b27c903898981ddd /source4/auth/kerberos | |
| parent | f92055f298a94ee53ed409f425198d0942238a8c (diff) | |
| download | samba-3fb5ae600efaeeb1cb7ee02d465b49bc52d111bf.tar.gz samba-3fb5ae600efaeeb1cb7ee02d465b49bc52d111bf.tar.bz2 samba-3fb5ae600efaeeb1cb7ee02d465b49bc52d111bf.zip | |
s4:auth/kerberos/kerberos_pac.c - fix another memory leak regarding the KRB principal
In addition fix a counter type
Autobuild-User: Matthias Dieter Wallnöfer <mdw@samba.org>
Autobuild-Date: Sat Dec 4 15:14:46 CET 2010 on sn-devel-104
Diffstat (limited to 'source4/auth/kerberos')
| -rw-r--r-- | source4/auth/kerberos/kerberos_pac.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/source4/auth/kerberos/kerberos_pac.c b/source4/auth/kerberos/kerberos_pac.c index 5e31c4596d..23b875056b 100644 --- a/source4/auth/kerberos/kerberos_pac.c +++ b/source4/auth/kerberos/kerberos_pac.c @@ -93,7 +93,7 @@ krb5_error_code check_pac_checksum(TALLOC_CTX *mem_ctx, DATA_BLOB modified_pac_blob; NTTIME tgs_authtime_nttime; krb5_principal client_principal_pac; - int i; + uint32_t i; krb5_clear_error_message(context); @@ -316,9 +316,12 @@ krb5_error_code check_pac_checksum(TALLOC_CTX *mem_ctx, if (!krb5_principal_compare_any_realm(context, client_principal, client_principal_pac)) { DEBUG(2, ("Name in PAC [%s] does not match principal name in ticket\n", logon_name->account_name)); + krb5_free_principal(context, client_principal_pac); return NT_STATUS_ACCESS_DENIED; } + krb5_free_principal(context, client_principal_pac); + #if 0 if (strcasecmp(logon_info->info3.base.account_name.string, "Administrator")== 0) { |