[IO-299] getPrefixLength returns null if filename has leading slashes

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.1, 2.1
• Fix Version/s: 2.5
• Component/s: Utilities
• Assignee: Unassigned
• Reporter: Rick Latrine
• Created: 17/Jan/12 09:57
• Updated: 08/Nov/16 17:59
• Resolved: 22/Apr/13 17:52

Description

Situation:
FilenameUtils.getPrefixLength is used in FilenameUtils.doNormalize.
FilenameUtils.normalize("////I don't want to become null!") returns null.

Problem:
Expected was: "I don't want to become null!"
The method FilenameUtils.getPrefixLength returns -1 for the mentioned string.
The root problem is found in following lines of code:

FilenameUtils.getPrefixLength

...
                int posUnix = filename.indexOf(UNIX_SEPARATOR, 2);
                int posWin = filename.indexOf(WINDOWS_SEPARATOR, 2);
                if ((posUnix == -1 && posWin == -1) || posUnix == 2 || posWin == 2) {
                    return -1;
                }
...

Solution:
All leading slashes should be ignored at all, but considering the rest of the string.