Description | User has MP3 file in issue 0015404, which contains 1MB of binary data in ID3 tag in text field, because of previous mess in combination of ID3v2.2 tags and ID3v2.3 tags. The crash was partially caused by fix of 0009311, where we try to read tags with ID3v2.2 name from ID3v2.3. This example shows, that it is not so safe, so we should better ignore such tags. It will lead to not reading tags from 0009311, but it should not crash. And we should add some length limitation when reading text field from ID3, so it will ignore text fields with more than e.g. 100000 chars. With exception for lyrics and comments, where we expect possible larger data by design. |
---|