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;
