Break out the that we request from principal from the entry and pass
it in as a separate argument. git-svn-id: svn://svn.h5l.se/heimdal/trunk/heimdal@17310 ec53bebd-3082-4978-b11e-865c3cabbd6b
This commit is contained in:
@@ -43,13 +43,12 @@ kadm5_s_delete_principal(void *server_handle, krb5_principal princ)
|
||||
hdb_entry_ex ent;
|
||||
|
||||
memset(&ent, 0, sizeof(ent));
|
||||
ent.entry.principal = princ;
|
||||
ret = context->db->hdb_open(context->context, context->db, O_RDWR, 0);
|
||||
if(ret) {
|
||||
krb5_warn(context->context, ret, "opening database");
|
||||
return ret;
|
||||
}
|
||||
ret = context->db->hdb_fetch(context->context, context->db,
|
||||
ret = context->db->hdb_fetch(context->context, context->db, princ,
|
||||
HDB_F_DECRYPT, &ent);
|
||||
if(ret == HDB_ERR_NOENTRY)
|
||||
goto out2;
|
||||
@@ -64,7 +63,7 @@ kadm5_s_delete_principal(void *server_handle, krb5_principal princ)
|
||||
|
||||
kadm5_log_delete (context, princ);
|
||||
|
||||
ret = context->db->hdb_remove(context->context, context->db, &ent);
|
||||
ret = context->db->hdb_remove(context->context, context->db, princ);
|
||||
out:
|
||||
hdb_free_entry(context->context, &ent);
|
||||
out2:
|
||||
|
@@ -1,5 +1,5 @@
|
||||
/*
|
||||
* Copyright (c) 1997 - 2005 Kungliga Tekniska H<>gskolan
|
||||
* Copyright (c) 1997 - 2006 Kungliga Tekniska H<>gskolan
|
||||
* (Royal Institute of Technology, Stockholm, Sweden).
|
||||
* All rights reserved.
|
||||
*
|
||||
@@ -75,11 +75,10 @@ kadm5_s_get_principal(void *server_handle,
|
||||
hdb_entry_ex ent;
|
||||
|
||||
memset(&ent, 0, sizeof(ent));
|
||||
ent.entry.principal = princ;
|
||||
ret = context->db->hdb_open(context->context, context->db, O_RDONLY, 0);
|
||||
if(ret)
|
||||
return ret;
|
||||
ret = context->db->hdb_fetch(context->context, context->db,
|
||||
ret = context->db->hdb_fetch(context->context, context->db, princ,
|
||||
HDB_F_DECRYPT, &ent);
|
||||
context->db->hdb_close(context->context, context->db);
|
||||
if(ret)
|
||||
|
@@ -347,12 +347,12 @@ kadm5_log_replay_delete (kadm5_server_context *context,
|
||||
krb5_storage *sp)
|
||||
{
|
||||
krb5_error_code ret;
|
||||
hdb_entry_ex ent;
|
||||
krb5_principal principal;
|
||||
|
||||
krb5_ret_principal (sp, &ent.entry.principal);
|
||||
krb5_ret_principal (sp, &principal);
|
||||
|
||||
ret = context->db->hdb_remove(context->context, context->db, &ent);
|
||||
krb5_free_principal (context->context, ent.entry.principal);
|
||||
ret = context->db->hdb_remove(context->context, context->db, principal);
|
||||
krb5_free_principal (context->context, principal);
|
||||
return ret;
|
||||
}
|
||||
|
||||
@@ -434,7 +434,7 @@ kadm5_log_replay_rename (kadm5_server_context *context,
|
||||
{
|
||||
krb5_error_code ret;
|
||||
krb5_principal source;
|
||||
hdb_entry_ex source_ent, target_ent;
|
||||
hdb_entry_ex target_ent;
|
||||
krb5_data value;
|
||||
off_t off;
|
||||
size_t princ_len, data_len;
|
||||
@@ -464,8 +464,7 @@ kadm5_log_replay_rename (kadm5_server_context *context,
|
||||
krb5_free_principal (context->context, source);
|
||||
return ret;
|
||||
}
|
||||
source_ent.entry.principal = source;
|
||||
ret = context->db->hdb_remove (context->context, context->db, &source_ent);
|
||||
ret = context->db->hdb_remove (context->context, context->db, source);
|
||||
krb5_free_principal (context->context, source);
|
||||
return ret;
|
||||
}
|
||||
@@ -553,9 +552,8 @@ kadm5_log_replay_modify (kadm5_server_context *context,
|
||||
return ret;
|
||||
|
||||
memset(&ent, 0, sizeof(ent));
|
||||
ent.entry.principal = log_ent.entry.principal;
|
||||
log_ent.entry.principal = NULL;
|
||||
ret = context->db->hdb_fetch(context->context, context->db,
|
||||
log_ent.entry.principal,
|
||||
HDB_F_DECRYPT, &ent);
|
||||
if (ret)
|
||||
goto out;
|
||||
|
@@ -50,11 +50,11 @@ modify_principal(void *server_handle,
|
||||
return KADM5_UNK_POLICY;
|
||||
|
||||
memset(&ent, 0, sizeof(ent));
|
||||
ent.entry.principal = princ->principal;
|
||||
ret = context->db->hdb_open(context->context, context->db, O_RDWR, 0);
|
||||
if(ret)
|
||||
return ret;
|
||||
ret = context->db->hdb_fetch(context->context, context->db, 0, &ent);
|
||||
ret = context->db->hdb_fetch(context->context, context->db,
|
||||
princ->principal, 0, &ent);
|
||||
if(ret)
|
||||
goto out;
|
||||
ret = _kadm5_setup_entry(context, &ent, mask, princ, mask, NULL, 0);
|
||||
|
@@ -51,11 +51,10 @@ kadm5_s_randkey_principal(void *server_handle,
|
||||
kadm5_ret_t ret;
|
||||
|
||||
memset(&ent, 0, sizeof(ent));
|
||||
ent.entry.principal = princ;
|
||||
ret = context->db->hdb_open(context->context, context->db, O_RDWR, 0);
|
||||
if(ret)
|
||||
return ret;
|
||||
ret = context->db->hdb_fetch(context->context, context->db, 0, &ent);
|
||||
ret = context->db->hdb_fetch(context->context, context->db, princ, 0, &ent);
|
||||
if(ret)
|
||||
goto out;
|
||||
|
||||
|
@@ -42,16 +42,17 @@ kadm5_s_rename_principal(void *server_handle,
|
||||
{
|
||||
kadm5_server_context *context = server_handle;
|
||||
kadm5_ret_t ret;
|
||||
hdb_entry_ex ent, ent2;
|
||||
hdb_entry_ex ent;
|
||||
krb5_principal remove = NULL;
|
||||
|
||||
memset(&ent, 0, sizeof(ent));
|
||||
ent.entry.principal = source;
|
||||
if(krb5_principal_compare(context->context, source, target))
|
||||
return KADM5_DUP; /* XXX is this right? */
|
||||
ret = context->db->hdb_open(context->context, context->db, O_RDWR, 0);
|
||||
if(ret)
|
||||
return ret;
|
||||
ret = context->db->hdb_fetch(context->context, context->db, 0, &ent);
|
||||
ret = context->db->hdb_fetch(context->context, context->db,
|
||||
source, 0, &ent);
|
||||
if(ret){
|
||||
context->db->hdb_close(context->context, context->db);
|
||||
goto out;
|
||||
@@ -80,12 +81,12 @@ kadm5_s_rename_principal(void *server_handle,
|
||||
}
|
||||
if(ret)
|
||||
goto out2;
|
||||
ent2.entry.principal = ent.entry.principal;
|
||||
remove = ent.entry.principal;
|
||||
ent.entry.principal = target;
|
||||
|
||||
ret = hdb_seal_keys(context->context, context->db, &ent.entry);
|
||||
if (ret) {
|
||||
ent.entry.principal = ent2.entry.principal;
|
||||
ent.entry.principal = remove;
|
||||
goto out2;
|
||||
}
|
||||
|
||||
@@ -95,11 +96,11 @@ kadm5_s_rename_principal(void *server_handle,
|
||||
|
||||
ret = context->db->hdb_store(context->context, context->db, 0, &ent);
|
||||
if(ret){
|
||||
ent.entry.principal = ent2.entry.principal;
|
||||
ent.entry.principal = remove;
|
||||
goto out2;
|
||||
}
|
||||
ret = context->db->hdb_remove(context->context, context->db, &ent2);
|
||||
ent.entry.principal = ent2.entry.principal;
|
||||
ret = context->db->hdb_remove(context->context, context->db, remove);
|
||||
ent.entry.principal = remove;
|
||||
out2:
|
||||
context->db->hdb_close(context->context, context->db);
|
||||
hdb_free_entry(context->context, &ent);
|
||||
|
Reference in New Issue
Block a user