diff options
| author | Volker Lendecke <vlendec@samba.org> | 2006-06-13 18:09:37 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:17:24 -0500 |
| commit | b369d0891afe8b777b837eaac317131232568ca7 (patch) | |
| tree | a795a4b39869ca3aca7cb6a6a9dc1dfb3cf4f943 /source/libads/krb5_setpw.c | |
| parent | 98751e8190317416de56b4a19a489c5f4b7d6bc9 (diff) | |
| download | samba-b369d0891afe8b777b837eaac317131232568ca7.tar.gz samba-b369d0891afe8b777b837eaac317131232568ca7.tar.xz samba-b369d0891afe8b777b837eaac317131232568ca7.zip | |
r16201: Fix Klocwork 439
Diffstat (limited to 'source/libads/krb5_setpw.c')
| -rw-r--r-- | source/libads/krb5_setpw.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source/libads/krb5_setpw.c b/source/libads/krb5_setpw.c index 254ca7b2a3a..ec2ff5afb1a 100644 --- a/source/libads/krb5_setpw.c +++ b/source/libads/krb5_setpw.c @@ -528,7 +528,6 @@ ADS_STATUS ads_krb5_set_password(const char *kdc_host, const char *princ, DEBUG(1,("Failed to parse kadmin/changepw (%s)\n", error_message(ret))); return ADS_ERROR_KRB5(ret); } - free(princ_name); /* parse the principal we got as a function argument */ ret = smb_krb5_parse_name(context, princ, &principal); @@ -537,9 +536,12 @@ ADS_STATUS ads_krb5_set_password(const char *kdc_host, const char *princ, krb5_free_principal(context, creds.server); krb5_free_context(context); DEBUG(1,("Failed to parse %s (%s)\n", princ_name, error_message(ret))); + free(princ_name); return ADS_ERROR_KRB5(ret); } + free(princ_name); + /* The creds.server principal takes ownership of this memory. Remember to set back to original value before freeing. */ orig_realm = *krb5_princ_realm(context, creds.server); |