Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Temp directory with space in the middle wrongly truncated from the rightmost space [CORE2151] #2582

Closed
firebird-issue-importer opened this issue Oct 25, 2008 · 6 comments

Comments

@firebird-issue-importer

Submitted by: @hvlad

Before Firebird 2.1 config parameter TempDirectories allows to specify temp directory size to use by Firebird.
I.e. entry in directory list might contain number separated by space, for example :

TempDirectories = c:\xxx 100; d:\yyy; d:\zzz 10000;

Therefore when this list was parsed rightmost space character was searched for and chars from this position up to the end of entry was removed (it was supposed that it is a number).

Starting from Firebird 2.1 this logic was abandoned but removal of characters from rightmost space to the end of directory entry was left in code

Commits: f5c1932 f6398ca

====== Test Details ======

Can't manupulate firebird.conf during tests run.

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Oct 25, 2008

Modified by: @hvlad

assignee: Vlad Khorsun [ hvlad ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Oct 25, 2008

Modified by: @hvlad

summary: Temp directory with space in the middle wrongly truncated at right side up to the space => Temp directory with space in the middle wrongly truncated from the rightmost space

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Oct 27, 2008

Modified by: @hvlad

status: Open [ 1 ] => Resolved [ 5 ]

resolution: Fixed [ 1 ]

Fix Version: 2.1.2 [ 10270 ]

Fix Version: 2.5 Beta 1 [ 10251 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 26, 2009

Modified by: @pcisar

status: Resolved [ 5 ] => Closed [ 6 ]

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Jan 19, 2016

Modified by: @pavel-zotov

QA Status: No test

@firebird-issue-importer
Copy link
Author

@firebird-issue-importer firebird-issue-importer commented Aug 22, 2016

Modified by: @pavel-zotov

status: Closed [ 6 ] => Closed [ 6 ]

QA Status: No test => Cannot be tested

Test Details: Can't manupulate firebird.conf during tests run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants