Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrong file length calculation for WavPack #180

Closed
lazka opened this issue Aug 11, 2014 · 2 comments
Closed

Wrong file length calculation for WavPack #180

lazka opened this issue Aug 11, 2014 · 2 comments
Labels
bug

Comments

@lazka
Copy link
Member

@lazka lazka commented Aug 11, 2014

Originally reported by: Marcus Habermehl (Bitbucket: bmh1980, GitHub: bmh1980)


Here I have some example files, included in the source code archive for taglib. One of these file is a WavPack file with no length. But mutagen calculates a length of 97391.54863945578.

#!python

bmh1980@deimos:/tmp/taglib-1.9.1/tests/data$ python
Python 2.7.6 (default, Mar 22 2014, 22:59:38) 
[GCC 4.8.2] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> import mutagen
>>> f = mutagen.File("no_length.wv")
4294967295 4294967295.0 75503649 44100
>>> f.info.length
97391.54863945578
>>> 
#!sh

bmh1980@deimos:/tmp/taglib-1.9.1/tests/data$ ls -l no_length.wv 
-rw-rw-r-- 1 bmh1980 bmh1980 532 Okt  8  2013 no_length.wv
bmh1980@deimos:/tmp/taglib-1.9.1/tests/data$ gst-discoverer-1.0 no_length.wv 
Analyzing file:///tmp/taglib-1.9.1/tests/data/no_length.wv
Done discovering file:///tmp/taglib-1.9.1/tests/data/no_length.wv

Topology:
  audio: Wavpack

Properties:
  Duration: 0:00:00.000000000
  Seekable: yes

bmh1980@deimos:/tmp/taglib-1.9.1/tests/data$

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Aug 11, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


Confirmed, thanks.

https://github.com/taglib/taglib/raw/master/tests/data/no_length.wv

@lazka

This comment has been minimized.

Copy link
Member Author

@lazka lazka commented Aug 12, 2014

Original comment by Christoph Reiter (Bitbucket: lazka, GitHub: lazka):


wavpack: Fix length calculation if sample count is missing in the header (Fixes issue #180)

@lazka lazka added minor bug labels Apr 7, 2016
@lazka lazka closed this Apr 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.