[dev] [cvs] commit: framework/Share/Share sql.php sql_hierarchical.php

Jan Schneider jan at horde.org
Fri Oct 24 14:02:19 UTC 2008


Zitat von Ronan SALMON <rsalmon at mbpgroup.com>:

> The last patches breaks kronolith,nag...
>
> We use ldap groups, so groups are strings like "cn=Domain  
> Users,ou=Groups,dc=home,dc=local"
>
> See below a quick fix that works down here.
> files: sql.php(674) and sql_hierarchical.php(226) :
>
> - $where .= ' OR (g.group_uid IN (' . implode(',', array_keys($groups)) . ')'
> -                    . ' AND (g.perm & ' . $perm .') != 0)';
> + $where .= ' OR (g.group_uid IN ("' . implode("','",  
> array_keys($groups)) . '")'
> +                    . ' AND (g.perm & ' . $perm .') != 0)';

Not quite, but fixed through using quote() instead.

Jan.

-- 
Do you need professional PHP or Horde consulting?
http://horde.org/consulting/



More information about the dev mailing list