jgerber wrote:

can you provide a link to the file you tried to upload.

Rainer: can you provide a link to the file you tried to upload?

This was reported by a user, not me. Andre asked him, if I recall correctly but they did not respond:

https://commons.wikimedia.org/wiki/Commons:Upload_help/Archive/2013/09#.22Unknown_error.22_when_uploading_audio_files_via_Upload_Wizard

https://commons.wikimedia.org/wiki/Commons:Help_desk/Archive/2013/09#Unable_to_upload_music_files_via_Upload_Wizard_and_Commons:_Upload

I've not seen any similar report recently.

I get the same error any time I try to upload an OGG audio file via https://commons.wikimedia.org/wiki/Special:Upload:

PHP fatal error in /srv/mediawiki/php-1.24wmf21/extensions/TimedMediaHandler/handlers/OggHandler/PEAR/File_Ogg/File/Ogg/Media.php line 119:
Allowed memory size of 268435456 bytes exhausted (tried to allocate 1802524276 bytes)

I tried files ranging in size from 13.2 MB to 6.1 MB. All failed with the same error.

The error seems to happen in _decodeBareCommentsHeader(), which parses the comments header in the ogg file. This header contains a series of comments related to the audio encoded in the stream as well as a string to identify the encoding software. The encoding software that I used was MediaHuman Audio Converter, version 1.9.1.

I think what's happening is that the function is choking when trying to parse the encoded COVERART comment, which is far larger than the other comments in the header.

Change 161174 had a related patch set uploaded by Kaldari:
Protect against exceeding memory errors when parsing Ogg files

https://gerrit.wikimedia.org/r/161174

For some reason, if I try to set any kind of size limit on the comments (even 100 MB), it causes several unit tests to fail. Anyone have any idea what's going on with those unit tests?
https://integration.wikimedia.org/ci/job/mwext-TimedMediaHandler-testextension/7/console

In the meantime, I was able to upload the Ogg file by removing the cover art from the file.

Got the unit tests passing with the fix. It's now skipping any comments that are longer than 10 KB, which should be more than enough for any of the information we're interested in.

Change 161174 merged by jenkins-bot:
Protect against exceeding memory errors when parsing Ogg files

https://gerrit.wikimedia.org/r/161174

Hopefully, this is fixed now.