oysteikt/heimdal
0
0
Fork 0
Code Issues 5 Pull Requests Releases Activity

gss/krb5: gsskrb5_acceptor_start authenticator leak (take two)

Browse Source 
Change-Id: I11be62ab806ea89258fe60e29e6d6488908070fa
This commit is contained in:
Jeffrey Altman
2016-11-19 23:59:30 -05:00
parent 2be6646adc
commit d9f7718a42
1 changed files with 4 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
lib/gssapi/krb5/accept_sec_context.c
View File

@@ -537,8 +537,8 @@ gsskrb5_acceptor_start(OM_uint32 * minor_status,
&ctx->flags, &ctx->flags,
&ctx->fwd_data); &ctx->fwd_data);
krb5_free_authenticator(context, &authenticator);
if (ret) { if (ret) {
krb5_free_authenticator(context, &authenticator);
return ret; return ret;
} }
} else { } else {
@@ -548,9 +548,8 @@ gsskrb5_acceptor_start(OM_uint32 * minor_status,
kret = krb5_crypto_init(context, kret = krb5_crypto_init(context,
ctx->auth_context->keyblock, ctx->auth_context->keyblock,
0, &crypto); 0, &crypto);
if(kret) { if (kret) {
krb5_free_authenticator(context, &authenticator); krb5_free_authenticator(context, &authenticator);
ret = GSS_S_FAILURE; ret = GSS_S_FAILURE;
*minor_status = kret; *minor_status = kret;
return ret; return ret;
@@ -564,10 +563,10 @@ gsskrb5_acceptor_start(OM_uint32 * minor_status,
kret = krb5_verify_checksum(context, kret = krb5_verify_checksum(context,
crypto, KRB5_KU_AP_REQ_AUTH_CKSUM, NULL, 0, crypto, KRB5_KU_AP_REQ_AUTH_CKSUM, NULL, 0,
authenticator->cksum); authenticator->cksum);
krb5_free_authenticator(context, &authenticator);
krb5_crypto_destroy(context, crypto); krb5_crypto_destroy(context, crypto);
if(kret) { if (kret) {
krb5_free_authenticator(context, &authenticator);
ret = GSS_S_BAD_SIG; ret = GSS_S_BAD_SIG;
*minor_status = kret; *minor_status = kret;
return ret; return ret;