php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login
Bug #67220 realpath() on MacOSX doesn't normalize the case of characters
Submitted: 2014-05-06 15:24 UTC Modified: 2015-08-10 20:24 UTC
Votes:3
Avg. Score:4.3 ± 0.5
Reproduced:3 of 3 (100.0%)
Same Version:2 (66.7%)
Same OS:3 (100.0%)
From: nicolas dot grekas+php at gmail dot com Assigned:
Status: Re-Opened Package: Filesystem function related
PHP Version: 5.5.12 OS: MacOSX/Darwin
Private report: No CVE-ID: None
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If this is not your bug, you can add a comment by following this link.
If this is your bug, but you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: nicolas dot grekas+php at gmail dot com
New email:
PHP Version: OS:

 

 [2014-05-06 15:24 UTC] nicolas dot grekas+php at gmail dot com
Description:
------------
See test script.

__FILE__, __DIR__, ReflectionClass::getFilename(), etc. are also affected (but getcwd() is OK).

Test script:
---------------
<?php

var_dump(realpath('/Users') === realpath('/users'));

Expected result:
----------------
bool(true)


Actual result:
--------------
bool(false)


Patches

Add a Patch

Pull Requests

Add a Pull Request

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2014-05-07 14:58 UTC] levim@php.net
-Status: Open +Status: Not a bug
 [2014-05-07 14:58 UTC] levim@php.net
Almost certainly this is because HFS+ is case-sensitive by default. Currently realpath "expands all symbolic links and resolves references to '/./', '/../' and extra '/' characters in the input path and returns the canonicalized absolute pathname".

Nowhere does that indicate anything about case sensitivity.
 [2014-05-07 15:06 UTC] rasmus@php.net
Well, I am still somewhat sympathetic to this though. The native OSX realpath() call does normalize the case. eg.

#include <stdio.h>
#include <stdlib.h>

int main(int argc, char **argv) {
        printf("%s %s\n",realpath("/Users",NULL),realpath("/users",NULL));
        return(0);
}

Outputs: /Users /Users

And we used to do so as well when we relied on the system-level realpath() call. However, after we optimized realpath() to allow us to do intra-path caching, we now use our own implementation.

Fixing this particular issue isn't that easy, and you are right, we don't promise to do path normalization in this function.
 [2014-05-07 15:14 UTC] nicolas dot grekas+php at gmail dot com
The doc says "returns the canonicalized absolute pathname"

"canonicalized" is what makes this a bug.

An other example that fails is require_once:

require_once 'TEST.PHP';
require_once 'test.php';

This does require the same file twice.
 [2014-05-07 18:10 UTC] tyrael@php.net
for the record HFS+ is optionally supports case sensitivity, but the current default is the case insensitive/preserve mode AFAIK.
I don't have a windows box at hand at the moment, but I'm fairly sure that realpath on windows does indeed correctly canonizes the path (http://www.php.net/realpath#example-2512 seems to be confirming this).

I think closing the issue with Not a bug is wrong, we should either keep it open until we can fix it, or mark as Won't fix, and document it.
 [2014-08-01 10:41 UTC] rullaf at gmail dot com
Please note this is sill an issue. Either the documentation should be changed to warn the realpath function does NOT return canonical path (it currently says `realpath — Returns canonicalized absolute pathname`) or the function should be fixed.
 [2015-08-04 15:06 UTC] bishop@php.net
I agree with tyrael: "I think closing the issue with Not a bug is wrong, we should either keep it open until we can fix it, or mark as Won't fix, and document it."

Side note, jpauli talks about the intra-path caching rasmus mentioned in this article: http://jpauli.github.io/2014/06/30/realpath-cache.html
 [2015-08-05 14:25 UTC] jpauli@php.net
-Status: Not a bug +Status: Re-Opened
 [2015-08-05 15:09 UTC] cmb@php.net
Automatic comment from SVN on behalf of cmb
Revision: http://svn.php.net/viewvc/?view=revision&amp;revision=337295
Log: added note regarding normalization of the character case on case-insensitive
filesystems (see #67220)
 [2015-08-05 15:26 UTC] andreas at heigl dot org
Has this issue been tested with both case-sensitive AND case-insensitive HFS+ and are the results the same?

Or is it only an issue with case-insensitive HFS+?
 [2015-08-10 20:24 UTC] tyrael@php.net
on case-sensitive HFS there is no issue, as there is nothing to "normalize".
HFS+ by default is case-insensitive case-preserving, the same as NTFS on windows by default.
given how we support returning the preserved version of the filename on Windows/NTFS and the fact that the realpath libc function on mac seems to properly return the preserved version(used https://gist.github.com/Tyrael/68dc7d6e0b96c2c231e4 to test it) I think it would be expected to work that way from php also.
 [2023-05-25 06:38 UTC] worldsaynews at gmail dot com
World Say News are sharing latest news about travel, tech, business, auto, home improvement, finance, entertainment etc. More info to visit: (https://worldsaynews.com)github.com
 
PHP Copyright © 2001-2024 The PHP Group
All rights reserved.
Last updated: Sat Apr 20 05:01:27 2024 UTC