svn commit: samba r15491 - in branches/SAMBA_4_0/source/heimdal/lib/gssapi: .

[abartlet at samba.org]

Author: abartlet
Date: 2006-05-07 15:02:27 +0000 (Sun, 07 May 2006)
New Revision: 15491

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

Log:
Always initialise is_cfx (found by Valgrind)

Always remember to free the crypto context (found by Luke Howard)


Modified:
   branches/SAMBA_4_0/source/heimdal/lib/gssapi/accept_sec_context.c


Changeset:
Modified: branches/SAMBA_4_0/source/heimdal/lib/gssapi/accept_sec_context.c
===================================================================
--- branches/SAMBA_4_0/source/heimdal/lib/gssapi/accept_sec_context.c	2006-05-07 13:40:56 UTC (rev 15490)
+++ branches/SAMBA_4_0/source/heimdal/lib/gssapi/accept_sec_context.c	2006-05-07 15:02:27 UTC (rev 15491)
@@ -77,6 +77,7 @@
 {
     krb5_keyblock *key;
     int acceptor = (context_handle->more_flags & LOCAL) == 0;
+    *is_cfx = 0;
 
     if (acceptor) {
 	if (context_handle->auth_context->local_subkey)
@@ -403,9 +404,10 @@
 					crypto, KRB5_KU_AP_REQ_AUTH_CKSUM, NULL, 0,
 					authenticator->cksum);
 	    krb5_free_authenticator(gssapi_krb5_context, &authenticator);
+	    krb5_crypto_destroy(gssapi_krb5_context, crypto);
 
 	    if(kret) {
-		ret = GSS_S_FAILURE;
+		ret = GSS_S_BAD_SIG;
 		*minor_status = kret;
 		gssapi_krb5_set_error_string ();
 		return ret;