On 9-Sep-04, at 12:48 PM, William A. Rowe, Jr. wrote:
You miss a case with this patch...
Consider a request for /content/foo/index.php/extra-path-info
The path doesn't end where you think it does, and directory walk would usually handle this case. Perhaps a patch to perform an extra sanity check on EACCES results is a better fix, at least introduced by a compile-time flag?
I'm sure you know the code better than I do. I thought that r->info.filetype would be 0 in this case, since extra-path-info doesn't exist; in which case the optimisation is already being skipped. But I might be completely out to lunch.
R,
