| Summary: | Multiple slash handling in Unix directory code wrong | ||
|---|---|---|---|
| Product: | LibreOffice | Reporter: | Chris Sherlock <chris.sherlock79> |
| Component: | LibreOffice | Assignee: | Chris Sherlock <chris.sherlock79> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | xiscofauli |
| Priority: | medium | ||
| Version: | Inherited From OOo | ||
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | target:5.4.0 | ||
| Crash report or crash signature: | Regression By: | ||
|
Description
Chris Sherlock
2017-05-20 07:51:56 UTC
It appears that the code comments are the issue here, they talk about double-slashes but actually, the code correctly checks for multiple slashes and changes to a single slash. I will add a unit test to check for this, and I've updated the comments. Chris Sherlock committed a patch related to this issue. It has been pushed to "master": http://cgit.freedesktop.org/libreoffice/core/commit/?id=0a72b799718a17b0ea49dc36d974da28000fc952 tdf#107968 - double slash handling misleading; add unit test It will be available in 5.4.0. The patch should be included in the daily builds available at http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: http://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback. Since there's a commit for this bug, moving it to NEW... The commit fixed the issue. |