Bug with MIDI type

[cstadler18]

Using latest release 1.9.6 (released Jun 03, 2013 : (with php 5.3.3) and Centos 6

http://www.midipedia.net/fichiers/midi/ ... is/Genesis Abacab.mid
Causes ID3 to just simply lock up and not return...
Help! Thanks, I also have a few examples from MP4 and MOV (Quicktime) that behave vary similarly let me know if you would like URLs for these also..

Thanks so much!
Craig

[James Heinrich]

I'll look at your sample file. Please also provide me samples of any other files that cause getID3 to hang.

[James Heinrich]

http://www.midipedia.net/fichiers/midi/ ... is/Genesis Abacab.mid

This file is truncated to 32kb (32768 bytes). The file header says there should be 22 tracks in the file but 32kB only gets about halfway through track #11, so only about half the file is there.

Nevertheless, getID3 should gracefully handle truncated files. This patch (around line 54 in module.audio.midi.php) should prevent it hanging and give a useful warning:

Code: Select all

while ((strlen($MIDIdata) - $offset) < 8) {
	if ($buffer = fread($this->getid3->fp, $this->getid3->fread_buffer_size())) {
		$MIDIdata .= $buffer;
	} else {
		$info['warning'][] = 'only processed '.($i - 1).' of '.$thisfile_midi_raw['tracks'].' tracks';
		$info['error'][] = 'Unabled to read more file data at '.ftell($this->getid3->fp).' (trying to seek to : '.$offset.'), was expecting at least 8 more bytes';
		return false;
	}
}

Full patched file attached, fix will be included in v1.9.7

[cstadler18]

http://walrus.wr.usgs.gov/tsunami/srila ... /pano1.mov also does the same. Thank you so much for your assistance and quick reply btw!!

[cstadler18]

Other media/urls that cause hangup :

MOV->

http://media.messe.ch/Messe/360/zurich-halle1.mov
http://walrus.wr.usgs.gov/tsunami/srilanka05/images/pano1.mov
http://www.akabak.net/38th_street.mov
http://video1.hobbico.com/gallery/dtx/dtxd14-360-view2.mov

MP4->
http://www.agapornis.narod.ru/video/fisheri_baby_birds3.3gp
http://www.mobile-review.com/review/image/sonyeric/t700/photo/video2.mp4
http://www.crosscolors.com/test/IndianaJones.mp4

Thank you!!

[cstadler18]

ASX -> (same problem examples)

http://www.alexisparkinn.com/photogalle ... om_air.wmv
http://previews.imineo.com/special-peche-carpe_.wmv
http://content.funny-base.com/videos2/s ... -magic.wmv

[James Heinrich]

http://previews.imineo.com/special-peche-carpe_.wmv

Parses fine (0.041 seconds).

http://www.alexisparkinn.com/photogalle ... om_air.wmv

Parses fine (0.672 seconds). Does have an extraordinary number of entries (15123) under [asf][header_extension_object][extension_data_parsed][4][stream_names] so may take some time to render the page if you're displaying the analysis in a browser (e.g. demo.browse)

http://content.funny-base.com/videos2/s ... -magic.wmv

Parses fine (0.681 seconds). Does have an extraordinary number of entries (15123) under [asf][header_extension_object][extension_data_parsed][4][stream_names] so may take some time to render the page if you're displaying the analysis in a browser (e.g. demo.browse)

The last two files are both from Metacafe, I guess their encoder does weird (but legal) encoding things to put such a huge pile of (blank) stream names in the header.

But all 3 files do parse fine, and in under a second on my machine.

[James Heinrich]

http://walrus.wr.usgs.gov/tsunami/srila ... /pano1.mov also does the same.

Parses fine (0.049 seconds). Quicktime VR panorama.

[James Heinrich]

Other media/urls that cause hangup

I looked at all 7 URLs provided and none of them cause getID3 to hang.

[cstadler18]

Thanks, recently we saw this one lock up... does it work for you?

http://previews.imineo.com/lucky-luke-phil-de-fer_.wmv

[James Heinrich]

Thanks, recently we saw this one lock up... does it work for you?

Works fine: File parsed in 0.051 seconds.