Bug 309693 - M4a tags are read wrong - composer tag is used for everything
Summary: M4a tags are read wrong - composer tag is used for everything
Status: RESOLVED NOT A BUG
Alias: None
Product: taglib
Classification: Frameworks and Libraries
Component: general (show other bugs)
Version: unspecified
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: Scott Wheeler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-11-07 13:13 UTC by Daniel Faust
Modified: 2012-11-07 15:16 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Daniel Faust 2012-11-07 13:13:44 UTC
When reading tags from any of my m4a files with any application that uses taglib, the content of the composer tag shows up in all tag fields.
It is completely reproducible and I tested if with taglib 1.7.2 and 1.8.0.
I can send you a file if you tell me where.

Reproducible: Always

Steps to Reproduce:
Try to read the tags of an .m4a file with the tagreader example application.
Actual Results:  
******************** "Kylie Minogue - Timebomb.m4a" ********************
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
-- TAG --
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
title   - "Paul Harris, Karen Poole & Matt Schwartz"
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
artist  - "Paul Harris, Karen Poole & Matt Schwartz"
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
album   - "Paul Harris, Karen Poole & Matt Schwartz"
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
year    - "0"
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
comment - "Paul Harris, Karen Poole & Matt Schwartz"
track   - "1"
TagLib: String::prepare() - Unicode conversion error.
TagLib: String::prepare() - Unicode conversion error.
genre   - "Paul Harris, Karen Poole & Matt Schwartz"
-- AUDIO --
bitrate     - 256
sample rate - 44100
channels    - 2
length      - 2:57

Expected Results:  
EasyTag shows the tags correctly:
Title: Timebomb
Artist: Kylie Minogue
Composer: Paul Harris, Karen Poole & Matt Schwartz
etc.
Comment 1 Lukáš Lalinský 2012-11-07 13:34:35 UTC
"TagLib: String::prepare() - Unicode conversion error." indicates that there is something wrong with the file. Can you send me an example to lalinsky@gmail.com?
Comment 2 Lukáš Lalinský 2012-11-07 13:48:19 UTC
I've tried both 1.7.2 and 1.8 and the attached file works fine for me:

lukas@gurgle:~/code/taglib$ ./examples/tagreader ~/Downloads/Kylie\ Minogue\ -\ Timebomb.m4a 
******************** "/home/lukas/Downloads/Kylie Minogue - Timebomb.m4a" ********************
-- TAG --
title   - "Timebomb"
artist  - "Kylie Minogue"
album   - "Timebomb [Single]"
year    - "2012"
comment - ""
track   - "1"
genre   - "Pop"
-- AUDIO --
bitrate     - 256
sample rate - 44100
channels    - 2
length      - 2:57

Which version of TagLib are you using? Are you sure it's not patched in some way?
Comment 3 Daniel Faust 2012-11-07 14:09:28 UTC
I use opensuse rpms. After compiling taglib from its sources the bug is gone.
Seems like I have to contact the packager.
Thanks for the fast answers and sorry for the noise.
Comment 4 Scott Wheeler 2012-11-07 14:35:51 UTC
Apparently OpenSUSE includes the RusXMMS patches to TagLib, which seems (a) like a bad idea and (b) is almost certainly what's causing this problem.
Comment 5 Daniel Faust 2012-11-07 15:16:45 UTC
Yes it is. I branched the package and removed the patch and it was the cause.
OpenSUSE bug report: https://bugzilla.novell.com/show_bug.cgi?id=788567