Re: [Gnump3d-devel] genre tag in mp3 files let gnump3d-index fail

[Martin Lohmeier]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Martin Lohmeier wrote:

| Ok, I had a look at the problem and everything is very strange. The line
| that cause the problem is the following:
|
| read $fh, my($bytes), $size - $hlen;
|
| $size is 2923585676 and $hlen is 10 and $size - $hlen should be
| 2923585666 ... why does perl think this is negativ???
|
| I could reproduce the problem in a small script I attached...I think
| this is a problem in perl...

The problem ist about read() and the buggy mp3 tag. The problem with
read: it outputs a wrong error message when it should read more then
2147483648 bytes ( 2Gbyte ). The correct error message should be "out of
memory" or better: "tryed to read more data then exist"

The problem with the buggy tag: for some reason $size is getting to big.
~ Something like

return unless $size < 1024000;

would solve the problem. 10240 should also be ok...

by, Martin


- --

Powered by Debian GNU / Linux
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org

iD8DBQFB8oarGxxhbGlQyAkRAte8AJ0XjKTUi1HokVd78Ckb5AMjkDs1ywCeKu0b
TvegU3br1hy1kzIqKYWtR2k=
=ptdY
-----END PGP SIGNATURE-----