Fri Jan 6 22:00:23 2006

Ticket #1590 (Closed: fixed)

Some mp3 audio clips will not load

Priority: normal Reporter: santiago
Severity: minor Assigned to: tomash
Component: gLiveSupport Status: closed
Version: 1.0 Resolution: fixed
Milestone: 1.1.x Keywords:  

Description by santiago:

Some mp3 clips will not load under Studio. Each clip appears to be consistent in Studio's ability to load it or not load it. Test data and sample clips are located in http://www.anemicaardvark.com/LiveSupport/

README.txt Truth_Table.ods Network.png A.mp3 B.mp3 C.mp3 and D.mp3

This problem appears to be in some way related to the ID3 tags.

Attachments

Changelog

Sat Jan 7 16:26:53 2006: Modified by fgerlits

  • component changed from LS-Knoppix Install CD to gLiveSupport
  • owner changed from douglas to fgerlits

Sat Jan 7 16:27:06 2006: Modified by fgerlits

  • status changed from new to assigned

Mon Jan 9 22:21:29 2006: Modified by fgerlits

  • status changed from assigned to new
  • owner changed from fgerlits to tomash

Tomas, could you take a look at this?

File A.MP3 has a version 2.3.0 id3 tag, and file C.MP3 has no id3 tag; these are uploaded OK. But files B.MP3 and D.MP3 both have version 2.4.0 id3 tags, and

  • they give an error message when trying to upload via xml-rpc (C++ gui);
  • no id3 tags are shown for them in the html gui.

I don't understand this; why are id3 tags even read by the storage server when I'm uploading with locstor.storeAudioClipOpen and ...Close? All the metadata reading is done by the C++ gui itself. (This may be a bad thing, in fact, but that's how it's done now.)

Here is the xml-rpc response from locstor.storeAudioClipClose: 

<value>
  <struct>
    <member>
      <name>faultCode</name>
      <value><i4>805</i4></value>
    </member>
    <member>
      <name>faultString</name>
      <value>ERROR:xrLocStor: 2 fread() [<a href='function.fread'>function.fread</a>]: Length parameter must be greater than 0. (/home/fgerlits/src/livesupport/src/modules/getid3/var/getid3.id3v2.php:91)</value>
    </member>
  </struct>
</value>

Wed Jan 18 17:41:27 2006: Modified by tomash

  • resolution set to fixed
  • status changed from new to closed

The getid3 library reports wrong format in these files with id3 tags v.2.4.0 (zero extraheader length). It's fixed by catching this warning.

There is only mime-type checking in storeAudioClipOpen method - not metadata parsing, but getid3 library do probably complete analyze of metadata part in audio file.

changeset: [1883]

Add/Change #1590 (Some mp3 audio clips will not load)




Change Properties






Action