Memory Leak in OggVorbis Decoder

[seife]

Hallo,

heute habe ich mir mal angeschaut, warum der Neutrino Audioplayer scheinbar Speicher leaked.

Anscheinend ist das ein Problem in der tremor-lib, und lässt sich durch updaten auf den aktuellen SVN Trunk Stand beheben.
Dummerweise gibt es bei xiph.org keine tarballs von Tremor, deswegen habe ich hier einen hingelegt: tremor-20090210.tar.bz2

Wenn den jemand irgendwo hosten könnte, dann könnte man das ins CDK einbauen und das OggVorbis abspielen vermutlich etwas zuverlässiger machen.

Edit:
Vorher:

Code: Alles auswählen

==18663== LEAK SUMMARY:
==18663==    definitely lost: 2,112 bytes in 36 blocks.
==18663==    indirectly lost: 64,784 bytes in 206 blocks.
==18663==      possibly lost: 157,398 bytes in 6,644 blocks.
==18663==    still reachable: 1,440,363 bytes in 9,894 blocks.
==18663==         suppressed: 8,160 bytes in 1 blocks.

Nachher:

Code: Alles auswählen

==20167== LEAK SUMMARY:
==20167==    definitely lost: 0 bytes in 0 blocks.
==20167==      possibly lost: 157,435 bytes in 6,645 blocks.
==20167==    still reachable: 1,439,725 bytes in 9,887 blocks.
==20167==         suppressed: 8,160 bytes in 1 blocks.

(das ist noch ganz ohne Abspielen, nur nach dem hinzufügen von 12 Ogg-Stücken zur Playlist, dann Beenden des Audioplayers)

[dbt]

Habs hier abgelegt, kannst du einsetzen.
http://www.tuxbox-cvs.sourceforge.net/c ... 10.tar.bz2

[rhabarber1848]

Probier bitte mal, ob dieser Tarball ausreicht:
http://ftp.debian.org/debian/pool/main/ ... rig.tar.gz

[seife]

dbt: danke, hab's damit eingechecked.
rhabarber: ja, das hätte auch gereicht, das einzige was sich seit der debian-Version noch geändert hat, ist irgendwas mit Windows Mobile

Wenn du willst, kannst du auf den debian-Tarball umstellen.

[jojo]

Der Compile läuft noch nicht so rund...

Code: Alles auswählen

( rm -rf tremor-20090210 || /bin/true ) && bunzip2 -cd Archive/tremor-20090210.tar.bz2 | TAPE=- tar -x && ( cd tremor-20090210; patch -p1 < ../Patches/tremor.diff ) && ((for f1 in config.guess config.sub; do (for f2 in `find tremor-20090210 -name $f1`; do (test -e $f2 && rm -f $f2 && ln -s /home/Playground/tuxbox-clean/tuxbox-cvs_K26/cdk/Patches/$f1 $f2 && echo "updated $f2") done) done) || /bin/true)
patching file vorbisidec.pc
cd tremor-20090210 && \
		AR=powerpc-tuxbox-linux-gnu-ar AS=powerpc-tuxbox-linux-gnu-as CC=powerpc-tuxbox-linux-gnu-gcc CXX=powerpc-tuxbox-linux-gnu-g++ NM=powerpc-tuxbox-linux-gnu-nm RANLIB=powerpc-tuxbox-linux-gnu-ranlib CFLAGS="-pipe -Os" CXXFLAGS="-pipe -Os" LDFLAGS="-Wl,-O1" PKG_CONFIG_PATH=/home/Playground/tuxbox-clean/dbox2_K26/cdkroot/lib/pkgconfig \
		./autogen.sh \
			--build=x86_64-suse-linux-gnu \
			--host=powerpc-tuxbox-linux-gnu \
			--prefix= && \
		make -j 1 && \
		perl -pi -e "s,^prefix=.*$,prefix=/home/Playground/tuxbox-clean/dbox2_K26/cdkroot," vorbisidec.pc && make install DESTDIR=/home/Playground/tuxbox-clean/dbox2_K26/cdkroot && /usr/bin/install -c -m644 vorbisidec.pc /home/Playground/tuxbox-clean/dbox2_K26/cdkroot/lib/pkgconfig && perl -pi -e "s,^libdir=.*$,libdir='/home/Playground/tuxbox-clean/dbox2_K26/cdkroot/lib',"  /home/Playground/tuxbox-clean/dbox2_K26/cdkroot/lib/libvorbisidec.la
checking for autoconf... 
checking for automake... yes
checking for libtool... libtoolize
Generating configuration files for vorbisdec, please wait....
  aclocal -I /opt/gnome/share/aclocal
  libtoolize --automake
  autoheader
  automake --add-missing 
configure.in:36: required file `../ltmain.sh' not found
make: *** [.deps/libvorbisidec] Error 1

[rhabarber1848]

configure.in:36: required file `../ltmain.sh' not found

@seife: Gab es einen Grund, warum Du diesen Patch rückgängig gemacht hast?
http://cvs.tuxbox-cvs.sourceforge.net/c ... 1.2&r2=1.3

Er muss wieder rein, dann läuft es auch mit libtool-2.x.
Wenn der Compile hier gut durchgelaufen ist, committe
ich meine Sachen, inkl. des libtool-Patches.

[rhabarber1848]

libtool problem fixed in CVS

[seife]

Er hatte nicht direkt applied (vermutlich wegen der Versionsnummer... ;() und bei mir ging's auch ohne. Ich dachte "das ist bestimmt in der neuen Version gefixt" , im SVN log stand auch was von "automake fixes".

Danke für's hinterherräumen.

[rhabarber1848]

Danke für's hinterherräumen.

Gern geschehen