Description Arvid Requate 2012-09-25 19:39:22 CEST

Die an Bug #28645 beobachtete Interpretation der LDAP-Filter in den Listener-Modulen könnte an den folgenden Zeilen in univention-directory-listener/src/filter.c liegen:

============================================================================
static int __cache_entry_ldap_filter_match(char* filter, int first, int last, CacheEntry *entry)
{
        /* sanity check */
        if (filter[first] != '(' || filter[last] != ')')
                return -1;
============================================================================

und

============================================================================
int cache_entry_ldap_filter_match(struct filter **filter, char *dn, CacheEntry *entry)
{
    /*...*/
    if (__cache_entry_ldap_filter_match((*f)->filter, 0, len-1, entry))
                        return 1;
============================================================================

Insgesamt sorgt das dafür, dass bei fehlschlagendem sanity check der Filter als gültig angesehen wird. Hier wäre zumindest eine zusätzliche Fehlermeldung sinnvoll.




+++ This bug was initially created as a clone of Bug #28645 +++