[AMQ-5714] Issue with pathCanonical() method within activemq startup for installation paths that consist of directories with a single character or 2 characters

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 5.11.1
• Fix Version/s: 5.12.0
• Assignee: Unassigned
• Reporter: James Spurin
• Created: 08/Apr/15 13:46
• Updated: 29/Nov/16 16:31
• Resolved: 10/Jul/15 13:50

Description

The pathCanonical method does not work as expected for directories that contain either single characters or 2 characters. This is caused by the following entry -

echo "${dst}" | sed -e 's#//#/#g' -e 's#/./#/#g' -e 's#/[^/]*/../#/#g'

This can be resolved with the following changes -

echo "${dst}" | sed -e 's#//#/#g' -e 's#/\./#/#g' -e 's#/[^/]*/\.\./#/#g'

Example, before and after -

# echo /1/2/3/4/11/22/33/44/.././x | sed -e 's#//#/#g' -e 's#/./#/#g' -e 's#/[^/]*/../#/#g'
/2/22/../x

# echo /1/2/3/4/11/22/33/44/.././x | sed -e 's#//#/#g' -e 's#/\./#/#g' -e 's#/[^/]*/\.\./#/#g'
/1/2/3/4/11/22/33/x

Issue Links

• is related to: AMQ-5503 Make init script more reliable
• links to: GitHub Pull Request #83