diff options
| author | Alexandra Ellwood <lxs@mit.edu> | 2008-10-17 15:45:05 +0000 |
|---|---|---|
| committer | Alexandra Ellwood <lxs@mit.edu> | 2008-10-17 15:45:05 +0000 |
| commit | fcd5639458992d23d49f1b867f405e53fb8388c9 (patch) | |
| tree | 4ae5d9936169cea6633018400b001b367a7f5169 | |
| parent | 077ccbfee3d205fe354bb1a7504fe8408d81c890 (diff) | |
| download | krb5-fcd5639458992d23d49f1b867f405e53fb8388c9.tar.gz krb5-fcd5639458992d23d49f1b867f405e53fb8388c9.tar.xz krb5-fcd5639458992d23d49f1b867f405e53fb8388c9.zip | |
Use a context to store error table info
Just sticking an einfo on the stack doesn't work because it's not
initialized. Also the detailed error message doesn't end up propogated back
to KIM.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20881 dc483132-0cff-0310-8789-dd5450dbe970
| -rw-r--r-- | src/kim/lib/kim_ui_plugin.c | 26 |
1 files changed, 18 insertions, 8 deletions
diff --git a/src/kim/lib/kim_ui_plugin.c b/src/kim/lib/kim_ui_plugin.c index 9b9675353..c307dd189 100644 --- a/src/kim/lib/kim_ui_plugin.c +++ b/src/kim/lib/kim_ui_plugin.c @@ -38,6 +38,7 @@ static const char *kim_ui_plugin_dirs[] = { LIBDIR "/krb5/plugins/kimui", NULL } struct kim_ui_plugin_context { + krb5_context kcontext; struct plugin_dir_handle plugins; struct kim_ui_plugin_ftable_v0 *ftable; void **ftables; @@ -56,6 +57,9 @@ static void kim_ui_plugin_context_free (kim_ui_plugin_context *io_context) if (PLUGIN_DIR_OPEN (&(*io_context)->plugins)) { krb5int_close_plugin_dirs (&(*io_context)->plugins); } + if ((*io_context)->kcontext) { + krb5_free_context ((*io_context)->kcontext); + } free (*io_context); *io_context = NULL; } @@ -76,6 +80,10 @@ static kim_error kim_ui_plugin_context_allocate (kim_ui_plugin_context *out_cont } if (!err) { + err = krb5_error (NULL, krb5_init_context (&context->kcontext)); + } + + if (!err) { PLUGIN_DIR_INIT(&context->plugins); context->ftable = NULL; context->ftables = NULL; @@ -98,7 +106,6 @@ kim_error kim_ui_plugin_init (kim_ui_context *io_context) { kim_error err = KIM_NO_ERROR; kim_ui_plugin_context context = NULL; - struct errinfo einfo; if (!err && !io_context) { err = check_error (KIM_NULL_PARAMETER_ERR); } @@ -109,16 +116,19 @@ kim_error kim_ui_plugin_init (kim_ui_context *io_context) if (!err) { PLUGIN_DIR_INIT(&context->plugins); - err = krb5int_open_plugin_dirs (kim_ui_plugin_dirs, - kim_ui_plugin_files, - &context->plugins, &einfo); + err = krb5_error (context->kcontext, + krb5int_open_plugin_dirs (kim_ui_plugin_dirs, + kim_ui_plugin_files, + &context->plugins, + &context->kcontext->err)); } if (!err) { - err = krb5int_get_plugin_dir_data (&context->plugins, - "kim_ui_0", - &context->ftables, - &einfo); + err = krb5_error (context->kcontext, + krb5int_get_plugin_dir_data (&context->plugins, + "kim_ui_0", + &context->ftables, + &context->kcontext->err)); } if (!err && context->ftables) { |