Taglib crash with a flac file #7811

mixxxbot · 2022-08-22T22:49:57Z

Reported by: stragu
Date: 2015-01-16T01:40:03Z
Status: Fix Released
Importance: Critical
Launchpad Issue: lp1411479
Attachments: [log file](https://bugs.launchpad.net/bugs/1411479/+attachment/4299630/+files/log file), [file that makes mixxx crash while scanning.flac](https://bugs.launchpad.net/bugs/1411479/+attachment/4316915/+files/file that makes mixxx crash while scanning.flac), [thread apply all bt output](https://bugs.launchpad.net/bugs/1411479/+attachment/4377061/+files/thread apply all bt output), [Backtrace after mixxx crash on library scan](https://bugs.launchpad.net/bugs/1411479/+attachment/4404070/+files/Backtrace after mixxx crash on library scan), taglib-1.9.1-bug-308.diff

Hi there

I have had this problem for a while, and it has happened to me with versions 1.10.1, 1.11.0 and the master build I am using currently (mixxx_1.12.0-alpha-git5124~~master-git5124-ppa1~~trusty1_trusty_amd64.deb).

The issue is that, when I start a fresh database (by removing or renaming my current mixxxdb.sqlite), Mixxx crashes while scanning my (rather large) library. I noticed that it happened on the same folder every time (I sometimes can see what folder it was scanning when it got stuck, but other times the windows close straight away) , so I removed that folder, but then it started doing it on a different one, etc.

Because of that problem, I have been using the same database for a while, but I would like to start clean because I noticed some albums do not appear in the library, even after a rescan. (A problem I have been talking about on the forum: http://www.mixxx.org/forums/viewtopic.php?f=3&t=6575)

When it crashes, the only extra line in the terminal is: "Segmentation fault (core dumped)"

I used the command "gdb --eval-command=run mixxx" and I get the following extra lines when Mixxx crashes:

[Thread 0x7fffd3392700 (LWP 7112) exited]
Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7ffeff170700 (LWP 7120)]
__memcpy_sse2_unaligned ()
at ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S:36
36 ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S: No such file or directory.

I then use the command "thread apply all bt" in gdb, and I get the following:

Thread 34 (Thread 0x7ffefe96f700 (LWP 7121)):
#0  0x00007ffff1884bad in poll () at ../sysdeps/unix/syscall-template.S:81
mixxxdj/mixxx#4910  0x00007ffff7bbd9b8 in ?? ()
   from /usr/lib/x86_64-linux-gnu/libportaudio.so.2
mixxxdj/mixxx#4911  0x00007ffff7bbe5e0 in ?? ()
   from /usr/lib/x86_64-linux-gnu/libportaudio.so.2
mixxxdj/mixxx#4912  0x00007ffff2fcf182 in start_thread (arg=0x7ffefe96f700)
    at pthread_create.c:312
mixxxdj/mixxx#4913  0x00007ffff1891efd in clone ()
    at ../sysdeps/unix/sysv/linux/x86_64/clone.S:111

Thread 33 (Thread 0x7ffeff170700 (LWP 7120)):
#⁠0 __memcpy_sse2_unaligned ()
at ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S:36
#⁠1 0x00007ffff3f139aa in TagLib::ByteVector::replace(TagLib::ByteVector const&, TagLib::ByteVector const&) () from /usr/lib/x86_64-linux-gnu/libtag.so.1
#⁠2 0x00007ffff3ee5d69 in TagLib::ID3v2::SynchData::decode(TagLib::ByteVector const&) () from /usr/lib/x86_64-linux-gnu/libtag.so.1
#⁠3 0x00007ffff3ee541d in TagLib::ID3v2::FrameFactory::createFrame(TagLib::ByteVector const&, TagLib::ID3v2::Header*) const ()
from /usr/lib/x86_64-linux-gnu/libtag.so.1
#⁠4 0x00007ffff3ee984f in TagLib::ID3v2::Tag::parse(TagLib::ByteVector const&)

These tests were made with mixxx_1.12.0-alpha-git5124~master-git5124 on Ubuntu 14.04, on an Asus x201ep.
Attached is a log of when the crash happens (but I don't think it contains anything interesting).

The text was updated successfully, but these errors were encountered:

mixxxbot · 2022-08-22T22:49:58Z

Commented by: stragu
Date: 2015-01-16T01:40:03Z
Attachments: [log file](https://bugs.launchpad.net/mixxx/+bug/1411479/+attachment/4299630/+files/log file)

mixxxbot · 2022-08-22T22:49:59Z

Commented by: stragu
Date: 2015-02-11T05:41:21Z

I tried to scan the same music folder (a copy of it on an external hard drive) with a fresh install of 1.11.0 on a different computer (a Samsung N310 with Ubuntu Studio 14.04.1) and I get a crash with the following error message:
"mixxx crashed with SIGABRT in __libc_message()"

The scan stalled on a different track though, but I am not sure how relevant this is.

mixxxbot · 2022-08-22T22:49:59Z

Commented by: stragu
Date: 2015-02-11T06:28:07Z

I just gave it another go and it crashed on the same track, which is a similar behaviour.

If I put the track it crashes on in a folder, on its own, and use this folder as Mixxx's library, then rescan, it does indeed crash again.

Could there be something wrong with my files? Or is it likely to be something that Mixxx does not lilke in those files?
It happens with files in different formats, with nothing special in the file names, they play fine in different players (Totem, Clementine...), the tags don't look weird at all...

I tried renaming the file, or removing all tags, to no avail. It still crashes on that particular file.

mixxxbot · 2022-08-22T22:50:00Z

Commented by: stragu
Date: 2015-02-11T06:57:55Z
Attachments: [file that makes mixxx crash while scanning.flac](https://bugs.launchpad.net/mixxx/+bug/1411479/+attachment/4316915/+files/file that makes mixxx crash while scanning.flac)

Interestingly, still with the testing on my Samsung N310, I removed the album that was making it crash, and now it scanned the whole library without a problem!
So I can say that on that second computers, the scan does not crash on the same files that made the first computer crash! Very confusing.
I tested that particular album on the first computer though, and it does make the scan crash. So at least that particular album (the only one that makes Mixxx crash on the Samsung N310) also makes Mixxx crash on the first computer.

I am planning to do a clean OS install on the first computer (the one that I actually use for DJing, the Asus x201ep, currently with Mixxx 1.12.0-alpha (build master r5167)) and a clean stable Mixxx install and see how it goes. I will keep you posted of any development.

For the time being, here is one track from the album that made Mixxx crash on both computer.

mixxxbot · 2022-08-22T22:50:00Z

Commented by: daschuer
Date: 2015-02-11T07:42:18Z

Can confirm it on Ubuntu Precise.

An other reason for a quarantine process for reading tags ..

mixxxbot · 2022-08-22T22:50:01Z

Commented by: daschuer
Date: 2015-02-11T07:45:59Z

As well on Ubuntu Trusty

mixxxbot · 2022-08-22T22:50:02Z

Commented by: stragu
Date: 2015-02-11T11:49:05Z

OK, I have some more info.

On the first computer (Asus x201ep, Ubuntu 14.04, Mixxx 1.12.0-alpha (build master r5167)) I deleted the mixxxdb.sqlite to do a clean scan of the copy of my library on my external hard drive (the same folder that I used as a library on the Samsung N310) and it worked fine without that particular album (of which I attached that track above). I did not encounter any of the issues I listed in the bug description (a large number of problematic files).

My understanding is that there are two problems:
- the one with Mixxx always crashing when scanning the track I attached to this bug report;
- the one with Mixxx crashing when analysing several different files only when they are on my laptop hard drive (even though it is an exact copy of the library on the external hard drive).

mixxxbot · 2022-08-22T22:50:02Z

Commented by: stragu
Date: 2015-04-16T12:13:27Z
Attachments: [thread apply all bt output](https://bugs.launchpad.net/mixxx/+bug/1411479/+attachment/4377061/+files/thread apply all bt output)

I am now on a clean install of KXstudio 14.04 64bits, with the Mixxx PPA to have 1.11.0.

I just did a first scan of my library and I have the same issue, with another FLAC file, interestingly from the same album as the audio file I attached above.

The results are similar. When using the command:
gdb --eval-command=run mixxx

I get the following:
Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fffb1ffb700 (LWP 5072)]
__memcpy_sse2_unaligned () at ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S:36
36 ../sysdeps/x86_64/multiarch/memcpy-sse2-unaligned.S: No such file or directory.

And after that, if I use the following command inside dbg:
thread apply all bt

I get the attached output.

I hope this helps.
Please let me know if I can help in any way to get this fixed for 1.12.0! Maybe we can end up with something that would bypass the problematic files and report graphically which ones they are?
By the way, do you know what the issue might be with those problematic files?

mixxxbot · 2022-08-22T22:50:03Z

Commented by: vlada-dudr
Date: 2015-05-25T10:58:53Z
Attachments: [Backtrace after mixxx crash on library scan](https://bugs.launchpad.net/mixxx/+bug/1411479/+attachment/4404070/+files/Backtrace after mixxx crash on library scan)

I got similar problem. ChakraOS "Euler", Lenovo Thinkpad T520.

It crashes with:
Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7ffef3fff700 (LWP 19751)]
0x00007ffff072e638 in __memcpy_avx_unaligned () from /usr/lib/libc.so.6

When I am rescanning library with my main database (I am using it for my sets), mixxx rescans without problem, than the scanner says "Scanning for cover art (Safe to cancel)" and than it crashes with same error.

Both bts are in attached file.

And all the way around it throws this warning:
Warning [Thread (pooled)]: libpng warning: iCCP: known incorrect sRGB profile

tried today with git branch 1.12 and compiled with faad=1 and tuned=1

mixxxbot · 2022-08-22T22:50:03Z

Commented by: sblaisot
Date: 2015-05-28T23:06:58Z

Do you have opus files in your library ?
I wonder if it could be related to https://bugs.launchpad.net/mixxx/+bug/1458380

mixxxbot · 2022-08-22T22:50:04Z

Commented by: daschuer
Date: 2015-07-23T11:11:06Z

Can confirm with current master and current 1.12 running on Ubuntu Trusty

mixxxbot · 2022-08-22T22:50:05Z

Commented by: stragu
Date: 2015-07-24T04:29:33Z

I just wanted to say that I do not have this problem anymore on my system, I am now on KXStudio 14.04 64 bits and a clean scan of my library works fine, using 1.11 or the 1.12 from the beta PPA.

The only thing I can think of is that on my previous install, my library was in my encrypted home folder... Could that be why there was an issue?

mixxxbot · 2022-08-22T22:50:06Z

Commented by: daschuer
Date: 2015-07-24T06:51:50Z

Could that be why there was an issue

I am afraid not. Since I can reproduce it on an unencrypted drive.

It looks like the bug is somewhere inside taglib
On my 64 bit system it is
libtag1-vanilla 1.9.1-2

What is yours?

mixxxbot · 2022-08-22T22:50:06Z

Commented by: stragu
Date: 2015-07-24T09:02:17Z

It looks like the bug is somewhere inside taglib
On my 64 bit system it is
libtag1-vanilla 1.9.1-2

I've got libtag1-vanilla 1.9.1-2 too, so that doesn't look like it.

mixxxbot · 2022-08-22T22:50:07Z

Commented by: daschuer
Date: 2015-07-24T09:43:49Z

Strange. What else coud be the reason that it does not crash anymore?

mixxxbot · 2022-08-22T22:50:08Z

Commented by: daschuer
Date: 2015-07-24T09:49:54Z

Just filed a Taglib bug: taglib/taglib#635

mixxxbot · 2022-08-22T22:50:08Z

Commented by: daschuer
Date: 2015-07-24T10:26:26Z

Got an answere!

We see here a bug that is already fixed upstream in taglib.
taglib/taglib#309

@rryan: Can you give an update. You have cherry picked the fix into our buildserver repro. But the code is gone.
Is it time to Include Taglib into the Mixxx source tree or schould we provide a ppa containing the patched version?

mixxxbot · 2022-08-22T22:50:09Z

Commented by: forkotov02
Date: 2015-08-01T09:16:40Z
Attachments: taglib-1.9.1-bug-308.diff

PPA with taglib from git: https://launchpad.net/~forkotov02/+archive/ubuntu/ppa

See my bugreport https://bugs.launchpad.net/ubuntu/+source/taglib/+bug/1353988

mixxxbot · 2022-08-22T22:50:09Z

Commented by: daschuer
Date: 2015-08-01T22:02:32Z

Cool Thank you for the info.

I think this is the best solution for the Mixxx PPA, just adding our own taglib deb.

@ilya: do you use any special scrips or such to setup the ppa version. Is there something we can re-use?

@rryan: @nschloe: Can I help you to make it happen?

mixxxbot · 2022-08-22T22:50:10Z

Commented by: ywwg
Date: 2015-11-19T03:03:32Z

we have upgraded the version of taglib to be included with mixxx, maybe this bug has gone away in recent builds?

mixxxbot · 2022-08-22T22:50:11Z

Commented by: daschuer
Date: 2015-11-19T08:00:19Z

It will be gone for Mac and Windows, but it persists on Linux.
We need to advance the requirement to Taglib 1.10 and provide it along with Mixxx

mixxxbot · 2022-08-22T22:50:11Z

Commented by: daschuer
Date: 2015-11-21T09:49:07Z

@owen @RJ
It would be nice to know you opinion about fixing this issue for Linux.
Is advancing the requirement to taglib 1.10 a good idea?

mixxxbot · 2022-08-22T22:50:12Z

Commented by: rryan
Date: 2015-11-21T17:24:08Z

It's hard to tell what the scope of this bug (and other TagLib 1.9 bugs we see) are in terms of users affected. So I'm not really able to determine a priority for whether I should spend time working on this. In general I'm not opposed if we have a strong reason to believe it's worth it. If someone can provide me a source package for TagLib 1.10 that supports Wily, Precise and Trusty -- I can push it to our PPAs.

I have a general concern about publishing other libraries to our PPA. I think most likely we'll push the library once and then forget about it (we still have a lost and forgotten portaudiov19 in our lp:mixxx/mixxx PPA from the one time we did this in the past!). What happens in this case? Is the user stuck on our old version when their distro offers a newer package?

mixxxbot · 2022-08-22T22:50:12Z

Commented by: daschuer
Date: 2015-11-21T18:54:49Z

You find the taglib source in the taglib repro on github.
If we reuse the Debian files from the system packages. A user will receive the taglib package from our ppa, as long as the default packages have not been updated.
If we provide mixxx along with a suitable taglib package. We have no maintained issue.
The main work will be signing an upload a source package.
I can do this, if I have access to our ppa.

mixxxbot · 2022-08-22T22:50:13Z

Commented by: Pegasus-RPG
Date: 2015-11-21T19:16:45Z

@RJ: As long as the package naming convention is respected, the apt system will install the latest version of a given package regardless of which source supplies it. So in the case of an old PortAudio, people on the older version of the OS would have our PA version since it's named newer than what the OS supplies, but those that have upgraded will get the OS version since it's newer than ours. Again, package version numbering is critical to this working correctly. (https://www.debian.org/doc/debian-policy/ch-controlfields.html#s-f-Version)

mixxxbot · 2022-08-22T22:50:14Z

Commented by: rryan
Date: 2015-11-21T19:39:25Z

You find the taglib source in the taglib repro on github.

I don't see a Debian source package in their repository.

I'm asking for a Debian source package that I can sign with our Mixxx master key and dput to our PPA.

mixxxbot · 2022-08-22T22:50:14Z

Commented by: ywwg
Date: 2015-11-21T23:42:10Z

I believe deb does the right thing and will not hold a user back on an old version of a library if a newer one is available from a different repo. I think we can recommend taglib 1.10, but I wouldn't require it. My first preference is to file a bug with the ubuntu packagers and ask them to update taglib rather than try to roll our own.

mixxxbot · 2022-08-22T22:50:15Z

Commented by: daschuer
Date: 2015-11-22T12:06:06Z

Yes, we can ask them, since the serious falc crasher will effect almost all other media applications.
I experienced taglib related crasher in clementine and rhythembox as well, but they are less annoying there because of the caranteine process they use.

I am only afraid that this discussion my delay our release. So putting taglib 1.10 in out ppa in the meantime is my preferred solution.

I would really prefer to require taglib 1.10. It will be rely no fun to being blamed for 1.9 crashes in upcoming press reviews. This will also equalize the situation for all our target.

mixxxbot · 2022-08-22T22:50:15Z

Commented by: daschuer
Date: 2015-11-24T00:19:08Z

@rryan:
I have build a trusty taglib deb here:
https://launchpad.net/~daschuer/+archive/ubuntu/mixxx/+packages

mixxxbot · 2022-08-22T22:50:16Z

Commented by: daschuer
Date: 2015-11-24T22:43:43Z

The Taglib binary was successfully tested on Ubuntu Precise and Trusty

mixxxbot · 2022-08-22T22:50:17Z

Commented by: daschuer
Date: 2015-11-26T22:14:21Z

Here is an example how other apps handle this:
https://launchpad.net/~forkotov02/+archive/ubuntu/ppa
The provide also opus for Precise.

I think we should do the same. Any thoughts?

mixxxbot · 2022-08-22T22:50:17Z

Commented by: ywwg
Date: 2015-11-26T23:01:15Z

Daniel you've committed a change that requires taglib >=1.10 even in old versions of ubuntu, do you even know if old versions of ubuntu have access to that version?

mixxxbot · 2022-08-22T22:50:18Z

Commented by: daschuer
Date: 2015-11-27T07:37:09Z

I have don this by accident, but it is not bad at all. See my comments here:
7858be0

mixxxbot · 2022-08-22T22:50:19Z

Commented by: rryan
Date: 2015-12-22T00:21:32Z

Thanks for the package Daniel, hopefully I did this right:

https://launchpad.net/~mixxx/+archive/ubuntu/mixxxbetas/+packages

I uploaded it for Precise. Does it need to be explicitly uploaded for Trusty/Vivid/Wily/Xenial?

mixxxbot · 2022-08-22T22:50:19Z

Commented by: daschuer
Date: 2015-12-22T06:49:20Z

I would say yes, however the must be a chance to get around it.
The Precise package will work on all version but I have not managed, to auto-update it.
Is the a way to make a package without a target version?

mixxxbot · 2022-08-22T22:50:20Z

Commented by: rryan
Date: 2015-12-30T06:24:17Z

Received confirmation from a forum user that the PPA automatically upgraded their taglib package to 1.10.

Our mixxx/mixxx and mixxx/mixxxbetas PPA has the taglib 1.10 package.

mixxxbot · 2022-08-22T22:50:20Z

Commented by: daschuer
Date: 2015-12-30T08:13:45Z

Sorry, I cannot confirm.
My Trusty installation does not receive the Precise 1.10 libtag package.
We need to provide it for every single edition.

mixxxbot · 2022-08-22T22:50:21Z

Commented by: daschuer
Date: 2015-12-31T17:29:12Z

This issue is still pending!
@rryan would you mind to upload the taglib version for the other Ubuntu versions as well?
Otherwise all users of a recent Ubuntu will still suffer this bug.

Thank you.

mixxxbot · 2022-08-22T22:50:22Z

Commented by: daschuer
Date: 2015-12-31T17:43:30Z

Alternatively, I can do it, once I have write access ;-) 
Since this can be a party stopper, we should not delay a solution too long.

mixxxbot · 2022-08-22T22:50:22Z

Commented by: rryan
Date: 2016-01-01T04:23:33Z

Trusty, Utopic, Vivid, and Wily are up now

mixxxbot · 2022-08-22T22:50:23Z

Commented by: daschuer
Date: 2016-01-02T12:36:33Z

Confirmed

mixxxbot · 2022-08-22T22:50:24Z

Issue closed with status Fix Released.

mixxxbot added bug party stopper labels Aug 22, 2022

mixxxbot closed this as completed Aug 22, 2022

This was referenced Aug 24, 2022

Crash on "Enable auto DJ" #8115

Closed

Crash on library rescan #8116

Closed

Segfault library scanner faulty flac file #8166

Closed

mixxxbot transferred this issue from another repository Aug 24, 2022

mixxxbot added this to the 2.0.0 milestone Aug 24, 2022

Taglib crash with a flac file #7811

Taglib crash with a flac file #7811

Comments

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022

mixxxbot commented Aug 22, 2022