diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2006-05-07 15:02:27 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 14:05:41 -0500 |
| commit | e9237f96eff562071b9cda2b7a4ea372af54abc9 (patch) | |
| tree | e6dfd6d3a61b00fddd5d204671d7ebc09d540e4c | |
| parent | ece95aad3da31809369e8867493bffb5cb749993 (diff) | |
| download | samba-e9237f96eff562071b9cda2b7a4ea372af54abc9.tar.gz samba-e9237f96eff562071b9cda2b7a4ea372af54abc9.tar.bz2 samba-e9237f96eff562071b9cda2b7a4ea372af54abc9.zip | |
r15491: Always initialise is_cfx (found by Valgrind)
Always remember to free the crypto context (found by Luke Howard)
(This used to be commit 4b44355d42592f4acaae459c6ae09dd928f083b7)
| -rw-r--r-- | source4/heimdal/lib/gssapi/accept_sec_context.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source4/heimdal/lib/gssapi/accept_sec_context.c b/source4/heimdal/lib/gssapi/accept_sec_context.c index afca449c5c..50d150e57c 100644 --- a/source4/heimdal/lib/gssapi/accept_sec_context.c +++ b/source4/heimdal/lib/gssapi/accept_sec_context.c @@ -77,6 +77,7 @@ gsskrb5_is_cfx(gss_ctx_id_t context_handle, int *is_cfx) { 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 @@ gsskrb5_acceptor_start 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; |