On Friday 23 October 2009, Ruediger Pluem wrote:
> > -    cp = strchr(path, '(');
> > -    if (cp) {
> > +    cp = strrchr(path, '(');
> > +    cp2 = path + strlen(path) - 1;
> > +    if (cp && (*cp2 == ')')) {
> >          *cp++ = '\0';
> > -
> > -        if (!(cp2 = strchr(cp, ')'))) {
> > -            return "Invalid argument: no closing parenthesis";
> > -        }
> 
> No error message any longer for missing parenthesis?
> I doubt that filenames like /somewhere/somefile(something are
>  intended.
> 

The question is if such filenames should be allowed. But I guess it's 
enought to allow them in case the cache size is given, too. Fixed in 
r829362

Reply via email to