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

Unicode DeviceSQL strings in PDB are UTF-16-BE, not UTF-16-LE #13291

Closed
0xsuk opened this issue May 26, 2024 · 4 comments
Closed

Unicode DeviceSQL strings in PDB are UTF-16-BE, not UTF-16-LE #13291

0xsuk opened this issue May 26, 2024 · 4 comments

Comments

@0xsuk
Copy link

0xsuk commented May 26, 2024

Bug Description

Mixxx uses kaitai struct to extract data from rekordbox pdb files.
rekordbox data binary was reversed by another project called crate-digger.
Mixxx uses an old copy of crate-pdb.ksy file for parsing pdb files, which needs to be updated.

Deep-Symmetry/crate-digger#14

image
As shown in the image, Japanese characters are not properly displayed.

Version

No response

OS

No response

@0xsuk 0xsuk added the bug label May 26, 2024
@Swiftb0y
Copy link
Member

I think the title of that pull request you mentioned is backwards, it's actually utf16-le and was just previously assumed to be utf16-be. But you're right, the ksy definitions are outdated.

@Swiftb0y
Copy link
Member

Thanks for digging through the issue, notifying us and testing the fix. It will be part of the next mixxx release.

@dimashenme
Copy link

is the fix available in a branch that I can compile locally while waiting for the next release to come out?

@ronso0
Copy link
Member

ronso0 commented Aug 9, 2024

#13293 has been merged on June 9th so the fix will be in Mixxx 2.4.2

You can build the 2.4 branch (stable), or 2.5 (currently beta).

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

No branches or pull requests

4 participants