svn commit: samba r24066 - in branches: SAMBA_3_2/source/libads SAMBA_3_2_0/source/libads

[gd at samba.org]

Author: gd
Date: 2007-07-27 09:22:43 +0000 (Fri, 27 Jul 2007)
New Revision: 24066

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=24066

Log:
Fix memleak found by Volker. We don't leak keys now with MIT and Heimdal.

Guenther

Modified:
   branches/SAMBA_3_2/source/libads/kerberos_verify.c
   branches/SAMBA_3_2_0/source/libads/kerberos_verify.c


Changeset:
Modified: branches/SAMBA_3_2/source/libads/kerberos_verify.c
===================================================================
--- branches/SAMBA_3_2/source/libads/kerberos_verify.c	2007-07-27 09:15:13 UTC (rev 24065)
+++ branches/SAMBA_3_2/source/libads/kerberos_verify.c	2007-07-27 09:22:43 UTC (rev 24066)
@@ -285,6 +285,7 @@
 		if (ret == KRB5KRB_AP_ERR_TKT_NYV || 
 		    ret == KRB5KRB_AP_ERR_TKT_EXPIRED ||
 		    ret == KRB5KRB_AP_ERR_SKEW) {
+			krb5_free_keyblock(context, key);
 			break;
 		}
 

Modified: branches/SAMBA_3_2_0/source/libads/kerberos_verify.c
===================================================================
--- branches/SAMBA_3_2_0/source/libads/kerberos_verify.c	2007-07-27 09:15:13 UTC (rev 24065)
+++ branches/SAMBA_3_2_0/source/libads/kerberos_verify.c	2007-07-27 09:22:43 UTC (rev 24066)
@@ -285,6 +285,7 @@
 		if (ret == KRB5KRB_AP_ERR_TKT_NYV || 
 		    ret == KRB5KRB_AP_ERR_TKT_EXPIRED ||
 		    ret == KRB5KRB_AP_ERR_SKEW) {
+			krb5_free_keyblock(context, key);
 			break;
 		}