From 2c1c2006627de0862a0dcd9b72d86d6432aede27 Mon Sep 17 00:00:00 2001 From: Pierangelo Masarati Date: Tue, 8 Dec 2009 01:34:22 +0000 Subject: [PATCH] log about invalid URI (more about ITS#6421) --- servers/slapd/overlays/dynlist.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) 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; } -- 2.39.5