Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 12 years ago

#18768 closed defect (bug) (fixed)

WordPress may not return correct $home_path when installed in a subdirectory

Reported by: ptahdunbar's profile ptahdunbar Owned by: dd32's profile dd32
Milestone: 3.4 Priority: normal
Severity: normal Version: 3.2
Component: Filesystem API Keywords: has-patch 3.4-early
Focuses: Cc:


I've got a local install of WordPress set within a subdirectory of the root (like all the cool kids).

Here's the file path:

After installing and setting pretty permalinks, WordPress wasn't able to create the .htaccess file even though it had write access to the root folder.

I traced the bug down to get_home_path() where it trys to guess the WordPress directory by strpos() the script path against the relative wp path. Starting the comparison from left to right, it stopped at the first wp it found giving me the following home path:

Well, that's wrong. It should be:

Patch fixes this by using strrpos() instead which stops at the last occurrence of the needle.
Given that it's matching against $_SERVER['SCRIPT_FILENAME'], it's more logical to work from the script path of WordPress than the other way around which may or may not contain duplicate matches.

Attachments (1)

ticket.18768.diff (638 bytes) - added by ptahdunbar 13 years ago.

Download all attachments as: .zip

Change History (6)

#1 @volcanicpixels
13 years ago

  • Component changed from General to Filesystem

I just tested the patch - works great.

#2 @nacin
13 years ago

  • Keywords 3.4-early added
  • Milestone changed from Awaiting Review to Future Release
  • Owner set to dd32
  • Status changed from new to assigned
  • Version changed from 3.3 to 3.2

Looks good to me. Unit tests might be helpful for this. Assigning to dd32.

#3 @dd32
13 years ago

  • Milestone changed from Future Release to 3.4

A simple way to trigger this is to install into a folder of the same name as your username.

Verified, patch works.

#4 @dd32
13 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In [19697]:

Fix edge case in get_home_path() where the incorrect path may be returned. Props ptahdunbar. Fixes #18768

#5 @WraithKenny
12 years ago

See #20449 fails for /wp/wp-admin/options-permalink.php since wp-admin is the last occurrence.

Note: See TracTickets for help on using tickets.