diff options
| author | Nalin Dahyabhai <nalin@redhat.com> | 2013-05-23 19:39:22 -0400 |
|---|---|---|
| committer | Nalin Dahyabhai <nalin@redhat.com> | 2013-05-23 19:46:12 -0400 |
| commit | dd71353dad22dcb07d1c9e90c7c634aa78ec0f77 (patch) | |
| tree | f4bd984aba9002942bdc25c3ba41735243266cbe | |
| parent | 41c4bc8377c4656a985466de57d97eb026895a8c (diff) | |
| download | slapi-nis-dd71353dad22dcb07d1c9e90c7c634aa78ec0f77.tar.gz slapi-nis-dd71353dad22dcb07d1c9e90c7c634aa78ec0f77.tar.xz slapi-nis-dd71353dad22dcb07d1c9e90c7c634aa78ec0f77.zip | |
Clean up logging a little bit
| -rw-r--r-- | src/back-shr.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/src/back-shr.c b/src/back-shr.c index 7d9de85..ade1336 100644 --- a/src/back-shr.c +++ b/src/back-shr.c @@ -1,5 +1,5 @@ /* - * Copyright 2008,2010,2011,2012 Red Hat, Inc. + * Copyright 2008,2010,2011,2012,2013 Red Hat, Inc. * * This Program is free software; you can redistribute it and/or modify * it under the terms of the GNU General Public License as published by @@ -901,7 +901,7 @@ backend_shr_update_references_cb(const char *group, const char *set, struct format_ref_attr_list **inref_attr_list, *inref_attr; struct format_ref_attr_list_link *this_attr_link, *prev_attr_link; struct format_ref_attr_list_link *next_attr_link; - const char *ndn, *dn; + const char *ndn, *dn, *rel_attr_list; int i, j, k, l, disposition, buffer_flags, n_ref_attrs, scope; set_data = backend_data; @@ -923,41 +923,43 @@ backend_shr_update_references_cb(const char *group, const char *set, break; } } + rel_attr_list = backend_shr_get_rel_attr_list(set_data); if (cbdata->mods[i] == NULL) { slapi_log_error(SLAPI_LOG_PLUGIN, state->plugin_desc->spd_id, "no interesting reference-based " "changes for \"%s\"/\"%s\" " - "made in \"%s\" (%s not in %s)\n", + "made in \"%s\" (\"%s\" not in \"%s\")\n", set_data->group, set_data->set, slapi_entry_get_ndn(cbdata->e), cbdata->modlist ? cbdata->modlist : "", - backend_shr_get_rel_attr_list(set_data)); + rel_attr_list ? rel_attr_list : ""); return TRUE; } else { slapi_log_error(SLAPI_LOG_PLUGIN, state->plugin_desc->spd_id, "interesting reference-based " "changes for \"%s\"/\"%s\" " - "made in \"%s\" (%s in %s)\n", + "made in \"%s\" (\"%s\" in \"%s\")\n", set_data->group, set_data->set, slapi_entry_get_ndn(cbdata->e), cbdata->modlist ? cbdata->modlist : "", - backend_shr_get_rel_attr_list(set_data)); + rel_attr_list ? rel_attr_list : ""); } } else { + rel_attr_list = backend_shr_get_rel_attr_list(set_data); slapi_log_error(SLAPI_LOG_PLUGIN, state->plugin_desc->spd_id, "reference-based changes for " "\"%s\"/\"%s\" made in (\"%s\") " - "(%s in %s or empty)\n", + "(\"%s\" in list \"%s\" or list empty)\n", set_data->group, set_data->set, slapi_entry_get_ndn(cbdata->e), cbdata->modlist ? cbdata->modlist : "", - backend_shr_get_rel_attr_list(set_data)); + rel_attr_list ? rel_attr_list : ""); } /* For every entry in this set which refers to this entry using |