Learn how easy it is to sync an existing GitHub or Google Code repo to a SourceForge project! See Demo

Close

#362 Can't get email notification for a page with / in its name

Rendering
closed
Reini Urban
5
2012-10-11
2004-07-21
Dan F
No

One of my users registered to get page notifications
for "MovieLens/SearchProblems" and this caused the wiki
to be uneditable for anyone using IE 6 !!

When any user edited any page,

  • in Firefox it would show:

lib/stdlib.php:1137: Warning[2]: Unknown modifier 'a'

  • in IE 6.0 it would show an error saying (essentially)
    "can't apply XSL to this XML"

I removed that page notification from the user's
preferences, and the problem went away.

Here is a stack trace:

/export/scratch/apache/htdocs/grplens/foo/phpwiki/lib/WikiDB.php:772:glob_match
/export/scratch/apache/htdocs/grplens/foo/phpwiki/lib/editpage.php:281:save
/export/scratch/apache/htdocs/grplens/foo/phpwiki/lib/editpage.php:91:savepage
/export/scratch/apache/htdocs/grplens/foo/phpwiki/lib/main.php:747:editpage
/export/scratch/apache/htdocs/grplens/foo/phpwiki/lib/main.php:488:action_edit
/export/scratch/apache/htdocs/grplens/foo/phpwiki/lib/main.php:932:handleaction
/export/scratch/apache/htdocs/grplens/foo/phpwiki/lib/main.php:940:main
/export/scratch/apache/htdocs/grplens/foo/phpwiki/index.php:972:include

This turned out to be in this function in stdlib.php:

function glob_match ($glob, $against, $case_sensitive =
true) {
return preg_match('/' . glob_to_pcre($glob) .
($case_sensitive ? '/' : '/i'\ ), $against);
}

with these parameters:

glob MovieLens/SearchProblems against SandBox
case_sensitive 1

You can see that this function is probably not /-safe.

Discussion

  • Reini Urban
    Reini Urban
    2005-03-04

    Logged In: YES
    user_id=13755

    Good catch and tricky to fix.

    Fixed in current CVS.