[krbdev.mit.edu #8695] Resource leak in krb5_524_conv_principal()

Bean Zhang via RT rt-comment at KRBDEV-PROD-APP-1.mit.edu
Thu Jun 14 23:46:47 EDT 2018


Hi Team,

In krb5-1.16.1/src/lib/krb5/krb/conv_princ.c,
krb5_524_conv_principal() calls profile_get_string() which allocates memory and stores into temp_realm,
but later if (tmp_realm_len > REALM_SZ - 1), before "return KRB5_INVALID_PRINCIPAL", does not free the storage temp_realm points to.

The fix is simple, just call "profile_release_string(tmp_realm);" before "return KRB5_INVALID_PRINCIPAL".

Could someone help to take a look?

Thanks,
Bean



More information about the krb5-bugs mailing list