log about invalid URI (more about ITS#6421)

This commit is contained in:
Pierangelo Masarati 2009-12-08 01:34:22 +00:00
parent db852e4378
commit 2c1c200662

View File

@ -170,7 +170,7 @@ dynlist_is_dynlist_next( Operation *op, SlapReply *rs, dynlist_info_t *old_dli )
} }
static int 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; slap_overinst *on = (slap_overinst *)op->o_bd->bd_info;
dynlist_info_t *dli = (dynlist_info_t *)on->on_bi.bi_private; 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 ) ); assert( !BER_BVISEMPTY( oldf ) );
if ( oldf->bv_val[0] != '(' ) { 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; needBrackets = 2;
} }
@ -618,7 +620,7 @@ dynlist_prepare_entry( Operation *op, SlapReply *rs, dynlist_info_t *dli )
} else { } else {
struct berval flt; struct berval flt;
ber_str2bv( lud->lud_filter, 0, 0, &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 */ /* error */
goto cleanup; goto cleanup;
} }