[drupal-devel] [bug] file_check_location is not working on Windows 2000

[Wes Cowley]

Issue status update for http://drupal.org/node/10885

 Project:      Drupal
 Version:      cvs
 Component:    file system
 Category:     bug reports
 Priority:     critical
 Assigned to:  Anonymous
 Reported by:  ccourtne
 Updated by:   Wes Cowley
 Status:       patch

I hand applied the patch to my Linux based installation and it solved
the problem I was having with filemanager/attachment.  I assume it
would fix the same problem I had with upload.module since the symptoms
were identical.




Wes Cowley



Previous comments:
------------------------------------------------------------------------

September 16, 2004 - 09:46 : ccourtne

Currently file_check_location is always returning false when passed a
file as the $source path.  If a directory is passed in $soruce
file_check_location works.  I've tracked this down to realpath
returning an empty string when passed a relative path which points to a
file.  This probably goes unnoticed if you are using the file.api's to
construct relative paths to the druapl file directory since
check_location failing just causes the durpal file directory to get
added which is what you wanted in the first place.


Replacing this line
  $source = realpath(dirname($source));
with this
  $source = realpath(dirname($source));
fixes it on my windows box using PHP 4.3.8.


I don't have things set up to do patches on my windows boxes here at
work yet.  I'll submit a formal patch tonight.




------------------------------------------------------------------------

September 16, 2004 - 09:47 : ccourtne

Dag nabit.


Replace
  $source = realpath($source);
with
  $source = realpath(dirname($source));


Craig




------------------------------------------------------------------------

September 16, 2004 - 09:56 : Anonymous

Attachment: http://drupal.org/files/issues/file-inc-file_check_location.patch (639 bytes)

The sooner there are patches, the better it is...


I tested this approach on my server and it works.


Steef




------------------------------------------------------------------------

September 16, 2004 - 21:34 : ccourtne

Attachment: http://drupal.org/files/issues/checklocation.patch (666 bytes)

Here is the patch.




------------------------------------------------------------------------

February 22, 2005 - 20:30 : dan90 at drupal.org

This also fixed my file upload problems with some nonstandard paths
under Linux/apache, php 4.3.4 drupal 4.5.2. maybe we should remove
mention of windows 2000?




------------------------------------------------------------------------

March 1, 2005 - 13:32 : killes at www.drop.org

Still applies.




------------------------------------------------------------------------

April 18, 2005 - 07:10 : ec

Is this patch still to be use with actual Drupal 4.6.0 ? Thanks. eric