Wednesday, May 21, 2014

[389-commits] ldap/servers

ldap/servers/plugins/acl/acllas.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 4f14e7df8215e30d6dd1205638886a877f46a70c
Author: Mark Reynolds <mreynolds@redhat.com>
Date: Wed May 21 14:36:01 2014 -0400

Ticket 47670 - Aci warnings in error log

Bug Description: Some aci warnings are unneccesarily written to the
error log.

Fix Description: Lower the log messaages from SLAPI_LOG_FATAL to SLAPI_LOG_ACL

Patch provided by: chatfield(Thanks!)

https://fedorahosted.org/389/ticket/47670

Reviewed by: nhosoi(Thanks!)

diff --git a/ldap/servers/plugins/acl/acllas.c b/ldap/servers/plugins/acl/acllas.c
index 38bc9a1..2d0dae9 100644
--- a/ldap/servers/plugins/acl/acllas.c
+++ b/ldap/servers/plugins/acl/acllas.c
@@ -3653,7 +3653,7 @@ acllas__client_match_URL (struct acl_pblock *aclpb, char *n_clientdn, char *url
/* Check the scope */
if ( ludp->lud_scope == LDAP_SCOPE_SUBTREE ) {
if (!slapi_dn_issuffix(n_clientdn, ludp->lud_dn)) {
- slapi_log_error( SLAPI_LOG_FATAL, plugin_name,
+ slapi_log_error( SLAPI_LOG_ACL, plugin_name,
"acllas__client_match_URL: url [%s] scope is subtree but dn [%s] "
"is not a suffix of [%s]\n",
normed, ludp->lud_dn, n_clientdn );
@@ -3663,7 +3663,7 @@ acllas__client_match_URL (struct acl_pblock *aclpb, char *n_clientdn, char *url
char *parent = slapi_dn_parent (n_clientdn);

if (slapi_utf8casecmp ((ACLUCHP)parent, (ACLUCHP)ludp->lud_dn) != 0 ) {
- slapi_log_error( SLAPI_LOG_FATAL, plugin_name,
+ slapi_log_error( SLAPI_LOG_ACL, plugin_name,
"acllas__client_match_URL: url [%s] scope is onelevel but dn [%s] "
"is not a direct child of [%s]\n",
normed, ludp->lud_dn, parent );
@@ -3673,7 +3673,7 @@ acllas__client_match_URL (struct acl_pblock *aclpb, char *n_clientdn, char *url
slapi_ch_free_string(&parent);
} else { /* default */
if (slapi_utf8casecmp ( (ACLUCHP)n_clientdn, (ACLUCHP)ludp->lud_dn) != 0 ) {
- slapi_log_error( SLAPI_LOG_FATAL, plugin_name,
+ slapi_log_error( SLAPI_LOG_ACL, plugin_name,
"acllas__client_match_URL: url [%s] scope is base but dn [%s] "
"does not match [%s]\n",
normed, ludp->lud_dn, n_clientdn );


--
389 commits mailing list
389-commits@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-commits

No comments:

Post a Comment