Last modified: 2014-01-29 09:52:02 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T62554, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 60554 - PEAR File_Ogg reports ogg/flac having 0 second length if total sample count field missing
PEAR File_Ogg reports ogg/flac having 0 second length if total sample count f...
Status: NEW
Product: MediaWiki extensions
Classification: Unclassified
TimedMediaHandler (Other open bugs)
unspecified
All All
: Unprioritized normal (vote)
: ---
Assigned To: Michael Dale
https://upload.wikimedia.org/wikipedi...
:
Depends on:
Blocks: multimedia
  Show dependency treegraph
 
Reported: 2014-01-29 01:18 UTC by Bawolff (Brian Wolff)
Modified: 2014-01-29 09:52 UTC (History)
7 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Bawolff (Brian Wolff) 2014-01-29 01:18:26 UTC
Ogg/flac files are allowed to have the total sample count field set to unknown (aka 0). When this happens, File_Ogg (and by extension TMH) reports the flac file has a length of 0 seconds, which is wrong.

I think what needs to happen in these situations is to take the approx_granule_pos and divide by sample_rate.

----

To quote ogg/flac spec:

"Redundant fields in the STREAMINFO packet may be set to zero (indicating "unknown" in native FLAC), which also facilitates single-pass encoding. These fields are: the minimum and maximum frame sizes, the total samples count, and the MD5 signature. "Unknown" values for these fields will not prevent a compliant native FLAC or Ogg FLAC decoder from decoding the stream."


Example file: https://commons.wikimedia.org/wiki/File:Tim_Berners-Lee_-_Today_(flac_-sample_s16_-f_ogg).oga

See also: https://commons.wikimedia.org/w/index.php?title=Commons:Village_pump&oldid=115012285#FLAC_on_Apple_devices
Comment 1 Bawolff (Brian Wolff) 2014-01-29 01:31:37 UTC
> Example file:
> https://commons.wikimedia.org/wiki/File:Tim_Berners-Lee_-_Today_(flac_-
> sample_s16_-f_ogg).oga
> 

Actual link https://upload.wikimedia.org/wikipedia/commons/archive/1/16/20140124185340%21Tim_Berners-Lee_-_Today_%28flac_-sample_s16_-f_ogg%29.oga (Its an old version of that file now)

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links