From: Pierangelo Masarati <ando@openldap.org>
Date: Tue, 8 Dec 2009 01:34:22 +0000 (+0000)
Subject: log about invalid URI (more about ITS#6421)
X-Git-Tag: ACLCHECK_0~18
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=2c1c2006627de0862a0dcd9b72d86d6432aede27;p=openldap

log about invalid URI (more about ITS#6421)
---

diff --git a/servers/slapd/overlays/dynlist.c b/servers/slapd/overlays/dynlist.c
index 457a520047..acfc2b387b 100644
--- a/servers/slapd/overlays/dynlist.c
+++ b/servers/slapd/overlays/dynlist.c
@@ -170,7 +170,7 @@ dynlist_is_dynlist_next( Operation *op, SlapReply *rs, dynlist_info_t *old_dli )
 }
 
 static int
-dynlist_make_filter( Operation *op, struct berval *oldf, struct berval *newf )
+dynlist_make_filter( Operation *op, Entry *e, const char *url, struct berval *oldf, struct berval *newf )
 {
 	slap_overinst	*on = (slap_overinst *)op->o_bd->bd_info;
 	dynlist_info_t	*dli = (dynlist_info_t *)on->on_bi.bi_private;
@@ -184,6 +184,8 @@ dynlist_make_filter( Operation *op, struct berval *oldf, struct berval *newf )
 	assert( !BER_BVISEMPTY( oldf ) );
 
 	if ( oldf->bv_val[0] != '(' ) {
+		Debug( LDAP_DEBUG_ANY, "%s: dynlist, DN=\"%s\": missing brackets in URI=\"%s\" filter\n",
+			op->o_log_prefix, e->e_name.bv_val, url );
 		needBrackets = 2;
 	}
 
@@ -618,7 +620,7 @@ dynlist_prepare_entry( Operation *op, SlapReply *rs, dynlist_info_t *dli )
 		} else {
 			struct berval	flt;
 			ber_str2bv( lud->lud_filter, 0, 0, &flt );
-			if ( dynlist_make_filter( op, &flt, &o.ors_filterstr ) ) {
+			if ( dynlist_make_filter( op, rs->sr_entry, url->bv_val, &flt, &o.ors_filterstr ) ) {
 				/* error */
 				goto cleanup;
 			}