aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGreg Hudson <ghudson@mit.edu>2016-07-14 11:15:40 -0400
committerTom Yu <tlyu@mit.edu>2016-09-02 18:30:32 -0400
commitdbc0548cb76b5fc557c7aea70147c780321fae7f (patch)
tree7f261a84fdde7a503e368f39bf981de88e765f09
parentd065fd7505a90227bb9694de5c78d36d53191dfa (diff)
downloadkrb5-dbc0548cb76b5fc557c7aea70147c780321fae7f.zip
krb5-dbc0548cb76b5fc557c7aea70147c780321fae7f.tar.gz
krb5-dbc0548cb76b5fc557c7aea70147c780321fae7f.tar.bz2
Fix leak on error in libkadm5_srv initialization
In init_any(), free handle->params on failure. (cherry picked from commit a60ed5c039642ec0bd6cdefba89762c3455fb84c) ticket: 8453 version_fixed: 1.13.7
-rw-r--r--src/lib/kadm5/srv/server_init.c11
1 files changed, 11 insertions, 0 deletions
diff --git a/src/lib/kadm5/srv/server_init.c b/src/lib/kadm5/srv/server_init.c
index c0e0b78..ec771ee 100644
--- a/src/lib/kadm5/srv/server_init.c
+++ b/src/lib/kadm5/srv/server_init.c
@@ -237,6 +237,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
KADM5_CONFIG_IPROP_PORT)
if ((handle->params.mask & REQUIRED_PARAMS) != REQUIRED_PARAMS) {
+ kadm5_free_config_params(handle->context, &handle->params);
free_db_args(handle);
free(handle);
return KADM5_MISSING_CONF_PARAMS;
@@ -244,6 +245,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
if ((handle->params.mask & KADM5_CONFIG_IPROP_ENABLED) == KADM5_CONFIG_IPROP_ENABLED
&& handle->params.iprop_enabled) {
if ((handle->params.mask & IPROP_REQUIRED_PARAMS) != IPROP_REQUIRED_PARAMS) {
+ kadm5_free_config_params(handle->context, &handle->params);
free_db_args(handle);
free(handle);
return KADM5_MISSING_CONF_PARAMS;
@@ -252,6 +254,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
ret = krb5_set_default_realm(handle->context, handle->params.realm);
if (ret) {
+ kadm5_free_config_params(handle->context, &handle->params);
free_db_args(handle);
free(handle);
return ret;
@@ -260,6 +263,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
ret = krb5_db_open(handle->context, db_args,
KRB5_KDB_OPEN_RW | KRB5_KDB_SRV_TYPE_ADMIN);
if (ret) {
+ kadm5_free_config_params(handle->context, &handle->params);
free_db_args(handle);
free(handle);
return(ret);
@@ -267,6 +271,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
if ((ret = krb5_parse_name(handle->context, client_name,
&handle->current_caller))) {
+ kadm5_free_config_params(handle->context, &handle->params);
krb5_db_fini(handle->context);
free_db_args(handle);
free(handle);
@@ -274,6 +279,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
}
if (! (handle->lhandle = malloc(sizeof(*handle)))) {
+ kadm5_free_config_params(handle->context, &handle->params);
krb5_db_fini(handle->context);
free_db_args(handle);
free(handle);
@@ -287,6 +293,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
/* can't check the handle until current_caller is set */
ret = check_handle((void *) handle);
if (ret) {
+ kadm5_free_config_params(handle->context, &handle->params);
free_db_args(handle);
free(handle);
return ret;
@@ -296,6 +303,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
(handle->params.mask & KADM5_CONFIG_MKEY_FROM_KBD)
&& handle->params.mkey_from_kbd);
if (ret) {
+ kadm5_free_config_params(handle->context, &handle->params);
krb5_db_fini(handle->context);
free_db_args(handle);
free(handle);
@@ -304,6 +312,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
ret = kdb_init_hist(handle, handle->params.realm);
if (ret) {
+ kadm5_free_config_params(handle->context, &handle->params);
krb5_db_fini(handle->context);
free_db_args(handle);
free(handle);
@@ -312,6 +321,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
ret = k5_kadm5_hook_load(context,&handle->hook_handles);
if (ret) {
+ kadm5_free_config_params(handle->context, &handle->params);
krb5_db_fini(handle->context);
krb5_free_principal(handle->context, handle->current_caller);
free_db_args(handle);
@@ -321,6 +331,7 @@ kadm5_ret_t kadm5_init(krb5_context context, char *client_name, char *pass,
ret = init_pwqual(handle);
if (ret) {
+ kadm5_free_config_params(handle->context, &handle->params);
k5_kadm5_hook_free_handles(context, handle->hook_handles);
krb5_db_fini(handle->context);
krb5_free_principal(handle->context, handle->current_caller);