From: Joshua C. <jc...@nd...> - 2003-01-08 03:26:40
|
1/7/2003 3:50:05 AM, "Thijs Kinkhorst" <li...@ki...> wrote: >Hello Joshua, >Thanks for the patch. I've applied a modified form of it to the current >1.4.0 tree so that should show up within a day in the snapshots on the >site. > > > >Thijs Thanks for applying the patch. However, I do have concerns regarding the modified form of the patch, as it has been applied to the latest CVS version. The error now properly shows up if the timezones.cfg file is non-existant. However, it also shows a warning from PHP regarding not being able to open the file. This is the output currently (with paths obscured): Warning: fopen("../locale/timezones.cfg", "r") - No such file or directory in /path/to/squirrelmail/include/options/personal.php on line 124 As this warning necessarily displays paths from which a user can divine the location of the directory where the web server's root directory is on the filesystem, it seems rather dangerous. -Joshua |