[prev in list] [next in list] [prev in thread] [next in thread] 

List:       taglib-devel
Subject:    Re: Recent taglib memleak introduced in 9b995544e
From:       Tomaz Canabrava <tcanabrava () kde ! org>
Date:       2016-10-24 16:19:49
Message-ID: CACk01_x78mSa+O0azRwoedCuNg3s6SHuEZays4OZY0Oddajgiw () mail ! gmail ! com
[Download RAW message or body]

could be - I'll test it here,
I assumed that using kdesrc-build ( that compiles taglib ) I would be
running that version of taglib.

On Mon, Oct 24, 2016 at 5:58 PM, Scott Wheeler <wheeler@kde.org> wrote:

> I suspect what you're seeing is that the indexer is using the old version
> of TagLib, and the file introduced specifically is testing for that bug.  I
> don't think that's really a bug of the TagLib developer sources.
>
> -Scott
>
> > On Oct 24, 2016, at 13:56, Tomaz Canabrava <tcanabrava@kde.org> wrote:
> >
> > please cc to me as I'm not on the mailing-list.
> >
> > I'm not sure if this bug is here for a longer time, but a binary file
> introduced in 9b995544e ( toc_many_children.mp3 ) causes taglib to eat my
> whole memory ( 20gbs ) and my system is unresponsive.
> >
> > I know that this file is used only for testing purposes but as soon as I
> checked out the repository, baloo tried to index it and in the index it
> uses taglib to find out about mp3 information, and poof.
> >
> > I tried to run the tests files and they run fine
> > vlc also crashes trying to open the file (also using taglib)
> >
> > Tomaz
>
>

[Attachment #3 (text/html)]

<div dir="ltr"><div>could be - I&#39;ll test it here,<br></div>I assumed that using \
kdesrc-build ( that compiles taglib ) I would be running that version of \
taglib.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct \
24, 2016 at 5:58 PM, Scott Wheeler <span dir="ltr">&lt;<a \
href="mailto:wheeler@kde.org" target="_blank">wheeler@kde.org</a>&gt;</span> \
wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex">I suspect what you&#39;re seeing is that the indexer is \
using the old version of TagLib, and the file introduced specifically is testing for \
that bug.   I don&#39;t think that&#39;s really a bug of the TagLib developer \
sources.<br> <span class="HOEnZb"><font color="#888888"><br>
-Scott<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
&gt; On Oct 24, 2016, at 13:56, Tomaz Canabrava &lt;<a \
href="mailto:tcanabrava@kde.org">tcanabrava@kde.org</a>&gt; wrote:<br> &gt;<br>
&gt; please cc to me as I&#39;m not on the mailing-list.<br>
&gt;<br>
&gt; I&#39;m not sure if this bug is here for a longer time, but a binary file \
introduced in 9b995544e ( toc_many_children.mp3 ) causes taglib to eat my whole \
memory ( 20gbs ) and my system is unresponsive.<br> &gt;<br>
&gt; I know that this file is used only for testing purposes but as soon as I checked \
out the repository, baloo tried to index it and in the index it uses taglib to find \
out about mp3 information, and poof.<br> &gt;<br>
&gt; I tried to run the tests files and they run fine<br>
&gt; vlc also crashes trying to open the file (also using taglib)<br>
&gt;<br>
&gt; Tomaz<br>
<br>
</div></div></blockquote></div><br></div>



[prev in list] [next in list] [prev in thread] [next in thread]