Bug 309693

Summary:	M4a tags are read wrong - composer tag is used for everything
Product:	[Frameworks and Libraries] taglib	Reporter:	Daniel Faust <hessijames>
Component:	general	Assignee:	Scott Wheeler <wheeler>
Status:	RESOLVED NOT A BUG
Severity:	normal	CC:	lalinsky
Priority:	NOR
Version:	unspecified
Target Milestone:	---
Platform:	openSUSE
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

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