diff options
| author | Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de> | 2010-03-16 16:26:25 +0100 |
|---|---|---|
| committer | Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de> | 2010-03-16 17:11:48 +0100 |
| commit | 973001e91a3988f4338f88599acbf4ab8978e1c9 (patch) | |
| tree | 2fcefc17de9f4b3137a62c203f1e982b07903846 | |
| parent | bdc0372f8a6116b3be7cd8bd6d398c0f1929ae68 (diff) | |
| download | samba-973001e91a3988f4338f88599acbf4ab8978e1c9.tar.gz samba-973001e91a3988f4338f88599acbf4ab8978e1c9.tar.bz2 samba-973001e91a3988f4338f88599acbf4ab8978e1c9.zip | |
heimdal - free always "ctx->password" when it isn't needed anymore
"strdup" does always create a new object in the memory (through "malloc") which
needs to be freed if it isn't used anymore.
| -rw-r--r-- | source4/heimdal/lib/krb5/init_creds_pw.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/source4/heimdal/lib/krb5/init_creds_pw.c b/source4/heimdal/lib/krb5/init_creds_pw.c index b615f5340a..f32b4186a4 100644 --- a/source4/heimdal/lib/krb5/init_creds_pw.c +++ b/source4/heimdal/lib/krb5/init_creds_pw.c @@ -107,6 +107,8 @@ free_init_creds_ctx(krb5_context context, krb5_init_creds_context ctx) free (ctx->pre_auth_types); if (ctx->in_tkt_service) free(ctx->in_tkt_service); + if (ctx->password) + free(ctx->password); if (ctx->keytab_data) free(ctx->keytab_data); krb5_data_free(&ctx->req_buffer); @@ -1354,7 +1356,7 @@ krb5_init_creds_set_password(krb5_context context, const char *password) { if (ctx->password) - memset(ctx->password, 0, strlen(ctx->password)); + free(ctx->password); if (password) { ctx->password = strdup(password); if (ctx->password == NULL) { |