Rockbox

This is the bug/patch tracker for Rockbox. Click here for more information.

Quick links: Bugs · Patches · Feature requests · Rockbox frontpage

Tasklist

FS#8795 - Fault of the escape processing in Changelog

Attached to Project: Rockbox
Opened by Kenjiro Arai (MoonWolf) - Tuesday, 25 March 2008, 02:24 GMT+2
Last edited by Miika Pekkarinen (miipekk) - Sunday, 20 April 2008, 01:48 GMT+2
Task Type Patches
Category Database
Status Closed
Assigned To No-one
Player type All players
Severity Low
Priority Normal
Reported Version current build
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Private No

Details

It is tagcache.c that there is a problem.
A problem is in import/export of the tag containing double-quote or a backslash. It is reproducible.
When the worst, a database is broken and the whole Rockbox becomes unstable.

double-quote should be escaped by backslash +double-quote in a write_tag function.
Moreover, a thing when a backslash comes to the end of a tag is considered, and a backslash should also be escaped.

un-escape processing of read_tag was also modified according to it.

The above correction improves compatibility with the tool of a third party.
   tagcache.patch (0.9 KiB)
This task depends upon

Closed by  Miika Pekkarinen (miipekk)
Sunday, 20 April 2008, 01:48 GMT+2
Reason for closing:  Accepted
Comment by Dominik Riebeling (bluebrother) - Wednesday, 26 March 2008, 00:03 GMT+2
What "tool of a third party" are you referring to?
Comment by Kenjiro Arai (MoonWolf) - Thursday, 27 March 2008, 03:37 GMT+2
It uses for reading rating and the count of playback of music from database_changelog.txt, and writing in MediaMonkey with the independent script which has not been released yet.
The actual situation and the way of reading and writing of 'Database' are not document-ized.
database_changelog.txt is its only hope.

Loading...