bug in module.audio-video.quicktime.php

[Evgeny Moysevich]

There is a bug in module.audio-video.quicktime.php, it shows itself when memory_limit is disabled ($this->getid3->memory_limit < 0). In this case negative number of bytes is passed to the fread function. I added a check for the $this->getid3->memory_limit in two places in module.audio-video.quicktime.php and now it works for me. I attached modified file, but my IDE reformatted the code, so if you will check the diffs, you will see a lot of them.

[James Heinrich]

I'm not sure which version you've modified. Could you please, for clarity, supply your patch against the current version of module.audio-video.quicktime.php since there seems to be more differences than what you describe (and not just IDE reformatting).

[Evgeny Moysevich]

I modified version 1.9.8-20140511. I checked the current version and it seems that issue with memory_limit is already fixed there.

[James Heinrich]

Probably fixed in this commit:
https://github.com/JamesHeinrich/getID3 ... 45fb139b13