Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336257 - [dstore] leading file.searator in DSTORE_LOG_DIRECTORY not handled
Summary: [dstore] leading file.searator in DSTORE_LOG_DIRECTORY not handled
Status: RESOLVED FIXED
Alias: None
Product: Target Management
Classification: Tools
Component: RSE (show other bugs)
Version: 3.0.3   Edit
Hardware: Other other
: P3 normal (vote)
Target Milestone: 3.3 M6   Edit
Assignee: David McKnight CLA
QA Contact: Martin Oberhuber CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks: 336350 336351
  Show dependency tree
 
Reported: 2011-02-03 11:36 EST by Onno Van den Troost CLA
Modified: 2011-02-04 09:55 EST (History)
1 user (show)

See Also:

Attachments
patch to remote leading / from log directory (2.48 KB, patch)
2011-02-04 09:44 EST, David McKnight CLA 		no flags Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Onno Van den Troost CLA 2011-02-03 11:36:01 EST 
getUserPreferencesDirectory() checks for the DSTORE_LOG_DIRECTORY property to allow changing the preferences directory.
In this process, getUserPreferencesDirectory() ensures there's a trailing file.separator, but it does not check for a leading one. 
This can result in a double file.separator in the path as DSTORE_LOG_DIRECTORY is appended to _userPreferencesDirectory which has a trailing file.separator.

Please backport to 3.0.3 and 3.2.1
Comment 1 David McKnight CLA 2011-02-04 09:44:47 EST 
Created attachment 188321 [details]
patch to remote leading / from log directory
Comment 2 David McKnight CLA 2011-02-04 09:55:38 EST 
I've committed the fix to cvs.

I've opened bug 336350 for the 3.2.x backport and I've opend bug 336351 for the 3.0.x backport.