diff options
| author | Andreas Schneider <asn@samba.org> | 2018-08-09 16:02:16 +0200 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2018-08-11 01:49:16 +0200 |
| commit | dbdbd4875ecac3e7334750f46f1f494b7afe6628 (patch) | |
| tree | 9fbb8739dc90c365790643a231f8d2d775008b6e /source3/libads | |
| parent | 3d32c0263b072e19335eba1451840284409ecb61 (diff) | |
| download | samba-dbdbd4875ecac3e7334750f46f1f494b7afe6628.tar.gz | |
s3:libads: Fix memory leaks in ads_krb5_chg_password()
Found by covscan.
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13567
Pair-Programmed-With: Justin Stephenson <jstephen@redhat.com>
Signed-off-by: Andreas Schneider <asn@samba.org>
Signed-off-by: Justin Stephenson <jstephen@redhat.com>
Reviewed-by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'source3/libads')
| -rw-r--r-- | source3/libads/krb5_setpw.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/source3/libads/krb5_setpw.c b/source3/libads/krb5_setpw.c index bc96ac603b1..0418fec5ad3 100644 --- a/source3/libads/krb5_setpw.c +++ b/source3/libads/krb5_setpw.c @@ -224,6 +224,7 @@ static ADS_STATUS ads_krb5_chg_password(const char *kdc_host, krb5_get_init_creds_opt_free(context, opts); krb5_free_context(context); free(realm); + smb_krb5_free_addresses(context, addr); DEBUG(1,("ads_krb5_chg_password: asprintf fail\n")); return ADS_ERROR_NT(NT_STATUS_NO_MEMORY); } @@ -234,6 +235,7 @@ static ADS_STATUS ads_krb5_chg_password(const char *kdc_host, kerb_prompter, NULL, 0, chpw_princ, opts); krb5_get_init_creds_opt_free(context, opts); + smb_krb5_free_addresses(context, addr); SAFE_FREE(chpw_princ); SAFE_FREE(password); |